This is an archive of the discontinued LLVM Phabricator instance.

[clang-format][docs][NFC] Fix example for Allman brace breaking style
ClosedPublic

Authored by jkorous on Mar 4 2019, 5:22 PM.

Details

Summary

I assume the example is wrong as it's clearly missing line-breaks before braces. That's about my level of understanding and I am totally oblivious to any finer details.

I just ran the example through clang-format with .clang-format like this:

BreakBeforeBraces: Allman

Diff Detail

Repository
rL LLVM

Event Timeline

jkorous created this revision.Mar 4 2019, 5:22 PM
Herald added a project: Restricted Project. · View Herald TranscriptMar 4 2019, 5:22 PM
JDevlieghere accepted this revision.Mar 4 2019, 5:35 PM

LGTM. Good catch, left looks like K&R or something.

This revision is now accepted and ready to land.Mar 4 2019, 5:35 PM
This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptMar 4 2019, 5:44 PM