atirit added a comment.

@MyDeveloperDay I expect to see exactly that. `clang-format` currently does not 
respect `AfterEnum`, treating it as always `true`. This is why I changed 
`UnwrappedLineParser.cpp`.

The unit test is incorrect as the style used for the `clang-format` C(++) unit 
tests uses `BreakBeforeBraces: Attach`, which should result in `AfterEnum` 
being treated as false, as indicated by the docs I quoted a few messages back. 
The unit test expects `AfterEnum: true` behaviour from a style that should 
result in `AfterEnum: false` behaviour. This is why I changed the unit test.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D93938/new/

https://reviews.llvm.org/D93938

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to