[clang-format] [PR45942] [[nodiscard]] causes && to be miss interpreted as BinaryOper...
authormydeveloperday <mydeveloperday@gmail.com>
Tue, 19 May 2020 15:41:50 +0000 (16:41 +0100)
committermydeveloperday <mydeveloperday@gmail.com>
Tue, 19 May 2020 15:41:50 +0000 (16:41 +0100)
commit6189dd06ad4889dfbccf185f9d81e72fe9173a8a
treecda36f502778f1549a64c09c39b6b79b85af1baf
parent70fbbcdd3437e9890307ef23d7057c565f142c44
[clang-format] [PR45942] [[nodiscard]] causes && to be miss interpreted as BinaryOperators

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

With Chromium style (although that is not important) its just it defines PointerAligmment: Left

The following arguments `S&&` are formatted differently depending on if the class has an attribute between it and the class identifier

```
class S {
  S(S&&) = default;
};

class [[nodiscard]] S {
  S(S &&) = default;
};
```

The prescense of [[nodiscard]] between the `class/struct` and the `{` causes the `{` to be incorrectly seen as a `TT_FunctionLBrace` which in turn transforms all the && to be `TT_BinaryOperators` rather than `TT_PointerOrReference`, as binary operators other spacing rules come into play causing a miss format

This revision resolves this by allowing the parseRecord to consider the [[nodisscard]]

Reviewed By: Abpostelnicu

Subscribers: cfe-commits

Tags: #clang, #clang-format

Differential Revision: https://reviews.llvm.org/D80008
clang/lib/Format/UnwrappedLineParser.cpp
clang/unittests/Format/FormatTest.cpp