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