Hacker News Reader

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?

View on HN

I 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.

Oh 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.

Linux Kernel Coding Style - https://www.kernel.org/doc/Documentation/CodingStyle

Reply on news.ycombinator.com