[PATCH] D93938: [clang-format] Fixed AfterEnum handling

[Ally Tiritoglu via Phabricator via cfe-commits]

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