sylvestre.ledru added a comment.
Done in https://reviews.llvm.org/D23317
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
opilarium added a comment.
> Thanks. I am rebasing and updating the patch.
Waiting for you.
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sylvestre.ledru added a comment.
Thanks. I am rebasing and updating the patch.
I don't know if poiru is still around.
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listi
djasper added a comment.
I still believe it's wrong to care about this in particular, but I also don't
want to hold clang-format hostage for my own beliefs. Obviously, enough people
care and the costs are limited.
Comment at: include/clang/Format/Format.h:356
@@ +355,3 @@
+ /
opilarium added a comment.
Can we just accept this merge request? It's backward compatible and helps
Mozilla to adapt clang-format.
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bi
opilarium added a comment.
It's the huge deal for my company too. It would be nice if you reconsider.
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sylvestre.ledru added a comment.
To be clear, this is blocking Mozilla to use much more clang-format
https://reviews.llvm.org/D10807
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits