4 ms·
Slightly off-topic, but I couldn't help but notice as a programmer. I find the coding style of the patch kind of random. The if statements, some of them have br
by Rygu 12y ago
Slightly off-topic, but I couldn't help but notice as a programmer. I find the coding style of the patch kind of random. The if statements, some of them have braces and some don't. Without a pattern it seems.
What is this style called?
- Gracana 12y agoI only see one place where an if statement has curly braces where it doesn't need them. The rest of them only have curly braces if they're necessary.
- Rygu 12y agoOh right, I see the second if statement at block @@ -1533,6 +1544,26 @@ indeed threw me off. I would value consistency over necessity. The else-if at the same block looks weird and the use of new lines could also help the code to be more legible and maintainable. Anyway, sorry went off-topic.
- regd006 12y agoLinux Kernel Coding Style - https://www.kernel.org/doc/Documentation/CodingStyle https://www.kernel.org/doc/Documentation/CodingStyle