This is an archive of the discontinued LLVM Phabricator instance.

Add more examples to clang-format configuration
ClosedPublic

Authored by sylvestre.ledru on Mar 15 2017, 11:22 AM.

Diff Detail

Event Timeline

docs/ClangFormatStyleOptions.rst
671

Is this example valid? I don't see why clang-format formats this this way?

686

Same

BasedOnStyle: LLVM
BreakBeforeBinaryOperators: All

More example (JS)

Improve consistency on an example

Daniel, is that ok with you?

djasper accepted this revision.Mar 20 2017, 4:14 AM

Looks good, just minor remarks.

include/clang/Format/Format.h
974

I'd contract these to a single line.

1007

nit: The indent on the right is incorrect (i.e. should not be different).

This revision is now accepted and ready to land.Mar 20 2017, 4:14 AM
sylvestre.ledru marked an inline comment as done.