[all-commits] [llvm/llvm-project] 40e771: [clang-format][regression][PR47461] ifdef causes c...

MyDeveloperDay via All-commits all-commits at lists.llvm.org
Thu Sep 17 05:23:32 PDT 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: 40e771c1c0d33c687230111271060c2ba761269f
      https://github.com/llvm/llvm-project/commit/40e771c1c0d33c687230111271060c2ba761269f
  Author: mydeveloperday <mydeveloperday at gmail.com>
  Date:   2020-09-17 (Thu, 17 Sep 2020)

  Changed paths:
    M clang/lib/Format/FormatTokenLexer.cpp
    M clang/unittests/Format/FormatTest.cpp

  Log Message:
  -----------
  [clang-format][regression][PR47461] ifdef causes catch to be seen as a function

https://bugs.llvm.org/show_bug.cgi?id=47461

The following change {D80940} caused a regression in code which ifdef's around the try and catch block cause incorrect brace placement around the catch

```
  try
  {
  }
  catch (...) {
    // This is not a small function
    bar = 1;
  }
}
```

The brace after the catch will be placed on a newline

Reviewed By: curdeius

Differential Revision: https://reviews.llvm.org/D87291




More information about the All-commits mailing list