[llvm-bugs] [Bug 50317] New: BreakBeforeBinaryOperators incorrect namespace-id handling in requires-clause

[via llvm-bugs]

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

            Bug ID: 50317
           Summary: BreakBeforeBinaryOperators incorrect namespace-id
                    handling in requires-clause
           Product: clang
           Version: 12.0
          Hardware: PC
                OS: Linux
            Status: NEW
          Severity: enhancement
          Priority: P
         Component: Formatter
          Assignee: unassignedclangbugs at nondot.org
          Reporter: oleksandr.koval.dev at gmail.com
                CC: djasper at google.com, klimek at google.com,
                    llvm-bugs at lists.llvm.org

In template head and in function body, BreakBeforeBinaryOperators has
consistent behavior. However, in requires-clause, when
BreakBeforeBinaryOperators is not 'None', it puts break after namespace-id.

With BreakBeforeBinaryOperators: NonAssignment:
template <typename T =
              std::xxxxxxxxxxxxxxxxxxxxxxx()
              && std::yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy()>
requires std::xxxxxxxxxxxxxxxxxxxxxxx() && std::
    yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy()
void test() {
  const b = std::xxxxxxxxxxxxxxxxxxxxxxx()
            && std::yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy();
}

With BreakBeforeBinaryOperators: None everything is OK:
template <typename T =
              std::xxxxxxxxxxxxxxxxxxxxxxx() &&
              std::yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy()>
requires std::xxxxxxxxxxxxxxxxxxxxxxx() &&
    std::yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy()
void test() {
  const b = std::xxxxxxxxxxxxxxxxxxxxxxx() &&
            std::yyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyyy();
}

-- 
You are receiving this mail because:
You are on the CC list for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-bugs/attachments/20210512/39337dbb/attachment-0001.html>