Hi WebKit folks,

over several reviews, I have been saying that the following line is a
coding style violation:

firstVariable = secondVariable = 0;

For a concrete example, the computePreferredLogicalWidths uses the
following pattern:

minWidth = maxWidth = max<int>(minWidth, tableLogicalWidth.value());

My justification is that those are 2 statements and thus should be on
2 lines per "Each statement should get its own line.". Some people
consider that the previous rule doesn't apply to multiple assignments
on one line and that such code is fine by the book.

What do people think?

Thanks,
Julien
_______________________________________________
webkit-dev mailing list
[email protected]
https://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to