clang-format update

Patrick Williams patrick at stwcx.xyz
Thu May 28 21:26:46 AEST 2020


All,

There was an update to clang pushed into CI last week[1].  We are now
running clang-10 on the repository-level unit-tests.

There was one change to clang-format that added a new `BraceWrapping`
option called `AfterCaseLabel`.  If your `.clang-format` file doesn't
have it set to `true`, this new version will likely reformat your case
statements in a way you don't desire.

I have put a fix into the common `.clang-format` file in docs[2].  My
suggestion is that the maintainers sync their local .clang-format files
with the one in docs[3].

1. https://github.com/openbmc/openbmc-build-scripts/commit/80d0728e6d5d3d623cda26a1bfe93d2eac4a8d3e
2. https://github.com/openbmc/docs/commit/b6de1960d9b896f2b48e6b0c676035afa0e432a6
3. https://github.com/openbmc/docs/blob/b6de1960d9b896f2b48e6b0c676035afa0e432a6/style/cpp/.clang-format
-- 
Patrick Williams
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <http://lists.ozlabs.org/pipermail/openbmc/attachments/20200528/710c5892/attachment.sig>


More information about the openbmc mailing list