On 16.07.26 13:27, Shawn Rutledge via Development wrote:
Let’s not split the cases for brace alignment.

We already do:

   if (x) {

but

   void fun(T x)
   {

Your suggestions to indent the if condition more just makes the problem authors 
of multi-line if conditions already have even worse, because lines become even 
longer.

This is also not about having a matcher or not. A braces matcher will not tell 
you where in

  if (x~~~~~100
      y~~~~~100
      z~~~~~100

the if condition ends and the body of the then clause starts, but this does:

  if (x~~~~~100
      y~~~~~100
  {
      z~~~~~100

Besides, Gerrit doesn't have a brace matcher.

Thanks,
Marc


--
Marc Mutz <[email protected]><mailto:[email protected]> (he/his)
Principal Software Engineer

The Qt Company
Erich-Thilo-Str. 10 12489
Berlin, Germany
www.qt.io<http://www.qt.io>

Geschäftsführer: Mika Pälsi, Juha Varelius, Juha Puputti
Sitz der Gesellschaft: Berlin,
Registergericht: Amtsgericht Charlottenburg,
HRB 144331 B

Public
-- 
Development mailing list
[email protected]
https://lists.qt-project.org/listinfo/development

Reply via email to