MyDeveloperDay added a comment.

@klimek thanks for offering for me to make the decision. I think given the fact 
that blender has it in their style guide and that they are clearly working 
around clang-formats inability to handle this case, I'm minded to LGTM this 
revision

I hope you are ok with that. given that @mitchell-stellar is prepared to 
support it, and I'm happy enough that I could fix something with that I hope 
you don't mind.

@mitchell-stellar are you happy with the name `OnlyMultiLine`? the reason I ask 
is:

`AlwaysBreakBeforeMultilineStrings` uses just `MultiLine`

> BTDS_MultiLine (in configuration: MultiLine) Force break after template 
> declaration only when the following declaration spans multiple lines.

I think given this won't be a standard true/false parameter users having to 
remember the "Only" part seems unnecessary


Repository:
  rC Clang

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

https://reviews.llvm.org/D68296



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

Reply via email to