This is an archive of the discontinued LLVM Phabricator instance.

Update the example of BS_Stroustrup to match what is done by clang-format
ClosedPublic

Authored by sylvestre.ledru on Oct 22 2018, 12:22 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

krasimir requested changes to this revision.Oct 22 2018, 1:57 PM
krasimir added inline comments.
docs/ClangFormatStyleOptions.rst
904 ↗(On Diff #170474)

Looking into this, I think your update of the example below is correct, but the description should stay as-is:

Like ``Attach``, but break before function definitions,...

The Always break before braces should still apply to the BS_Allman option below.
Am I missing something here?

This revision now requires changes to proceed.Oct 22 2018, 1:57 PM

Remove the desc update

krasimir accepted this revision.Oct 24 2018, 7:37 AM

Thank you!

This revision is now accepted and ready to land.Oct 24 2018, 7:37 AM
This revision was automatically updated to reflect the committed changes.