MyDeveloperDay added a comment. In D33029#1976981 <https://reviews.llvm.org/D33029#1976981>, @bbassi wrote:
> @MyDeveloperDay hey, I am currently working on this, and adding a new option > called BreakBeforeClosingBracket. I have some questions to understand the > existing code, they might not be directly linked to this change so I am not > sure if this the best place to ask those questions. What do you think? e.g. I > don't understand what FakeLParens is and have some questions about that. @bbassi, you might want to consider starting a new revision if you are going to add a different option. Its probably best to discuss questions about adding that option in its own revision rather than confusing the conversation here. (you can always cross reference with a mention. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D33029/new/ https://reviews.llvm.org/D33029 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
