[PATCH] D117421: [clang-format] Fix incorrect alignment of operator= overloads.

Marek Kurdej via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Jan 17 00:52:18 PST 2022


curdeius added a comment.

With your patch I still see the bug in this case:

  struct S {
    constexpr S(const S &) = default;
    void f()               = default;
    S &operator/**/        =(S) {}
  };

Mind the `operator  =`.
It happens every time that `operator=` has only declaration (no definition body).

Another failure:

  struct S {
    /*comment*/ S(const S &) = default;
    void f()                 = default;
    S &operator/**/          =(S) {}
  };

As mentioned yesterday, comment between `operator` and `=`.

Note, `/*comment*/` is there just to make the spacing more visible. It's unrelated to the bug otherwise.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117421/new/

https://reviews.llvm.org/D117421



More information about the cfe-commits mailing list