djasper accepted this revision.
djasper added a comment.
This revision is now accepted and ready to land.
Yes, thank you!
https://reviews.llvm.org/D31408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
sylvestre.ledru added a comment.
Daniel, is that ok with you?
https://reviews.llvm.org/D31408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sylvestre.ledru added inline comments.
Comment at: docs/ClangFormatStyleOptions.rst:941
+ ContinuationIndentWidth: 2
+ ColumnLimit: 15
+
djasper wrote:
> You could force line wraps with tailing //-comments instead.
much better. thanks
sylvestre.ledru updated this revision to Diff 94008.
https://reviews.llvm.org/D31408
Files:
docs/ClangFormatStyleOptions.rst
include/clang/Format/Format.h
Index: docs/ClangFormatStyleOptions.rst
===
---
djasper added a comment.
Generally please upload diffs with more contexts. For some here it's not even
clear to which option they refer ;)
Comment at: docs/ClangFormatStyleOptions.rst:941
+ ContinuationIndentWidth: 2
+ ColumnLimit: 15
+
You could
sylvestre.ledru added a comment.
Daniel, wdyt? :)
https://reviews.llvm.org/D31408
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
sylvestre.ledru updated this revision to Diff 93195.
sylvestre.ledru added a comment.
Thanks, indeed! :)
https://reviews.llvm.org/D31408
Files:
docs/ClangFormatStyleOptions.rst
include/clang/Format/Format.h
Index: include/clang/Format/Format.h
Eugene.Zelenko added inline comments.
Comment at: docs/ClangFormatStyleOptions.rst:644
+ .. code-block:: c++
+
Shouldn't it be java?
Comment at: docs/ClangFormatStyleOptions.rst:1293
+ .. code-block:: c++
+
Shouldn't it
sylvestre.ledru created this revision.
Herald added a subscriber: klimek.
https://reviews.llvm.org/D31408
Files:
docs/ClangFormatStyleOptions.rst
include/clang/Format/Format.h
Index: include/clang/Format/Format.h
===
---