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

[Marcus Johnson via Phabricator via cfe-commits]

MarcusJohnson91 added a comment.

Not trying to take over this patch, because I've got a ton of patches I need to 
be finishing up myself.

but I think the problematic code is in UnwrappedLineParser.cpp:1900

  if (FormatTok->Tok.getKind() == ClosingBraceKind) {
    if (IsEnum && !Style.AllowShortEnumsOnASingleLine)
      addUnwrappedLine();
    nextToken();
    return !HasError;
  }

I'd change that if statement to

  if (FormatTok->Tok.getKind() == ClosingBraceKind) {
    if (IsEnum && (!Style.AllowShortEnumsOnASingleLine || 
!BraceWrapping.AfterEnum))
        addUnwrappedLine();
    nextToken();
    return !HasError;
  }

I haven't tried this code at all, just a gut feeling based on my experience 
fixing the extern block wrapping; hopefully it helps point you in the right 
direction.


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