clang-tidy: where does the comma get removed when removing an arg?

Clang Frontend
#1

If I create a removal FixItHint in clang-tidy to remove a function argument (that is, the CharSourceRange in the cal to FixItHint::CreateRemoval() spans a function-call argument), the associated comma for that argument (if any) is removed as well.

For example, when removing the second argument of foo(3, 4), clang-tidy (correctly) removes the comma to produce foo(3).

I’d like to understand where that happens in the code. I’ve looked through much of the code involved and searched the codebase for tok::comma, but was completely unable to discover the source of this modification.

thanks,
yitzhak

#2

I think the cleanup is done by cleanupAroundReplacements (clang-tidy formats the replacements before applying them).

#3

thanks, that’s quite helpful. For those curious, the code responsible is in clang/lib/Format/Format.cpp, in Cleaner::analyze().