MyDeveloperDay added a comment.

In D112019#3378024 <https://reviews.llvm.org/D112019#3378024>, 
@HazardyKnusperkeks wrote:

> Why limit to macros, could it be a member of AlignConsecutiveStyle and apply 
> to the other stuff as well?

I personally don't have a use case other than the resource.h case that I raise 
above, do you think this would be useful to be elsewhere in the other 
AlignConsecutive cases?

Of course, this code alters AlignMacroSequence() and not AlignTokens() what if 
we unified the options (so they were part of the struct? but kept the 
functionality change in separate reviews? for now the options would have no 
effect on other Consecutive options?


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D112019/new/

https://reviews.llvm.org/D112019

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to