Replies: 4 comments 11 replies
-
Of course, maybe the argument might be to go full LKML style, i.e. with |
Beta Was this translation helpful? Give feedback.
-
Interesting GLib has the same issue: [hughsie@fedora glib (master %)]$ git grep "sizeof (" | wc -l |
Beta Was this translation helpful? Give feedback.
-
It turns out not many people agree with my space before bracket, except for sizeof rule :) https://twitter.com/hughsient/status/1418957839503134725 |
Beta Was this translation helpful? Give feedback.
-
What about the option of no brackets at all for |
Beta Was this translation helpful? Give feedback.
-
@superm1 @djcampello
At the moment in the codebase we use both
sizeof(buf)
andsizeof (buf)
-- in my head the first makes the most sense as a it's a preprocessor directive, but on the other hand it just looks like a function to most people.What do we think about doing a bulk conversion of the mixed format we have now to either the former or the latter. I know it's baby steps, but I thought we could start with some of the lowest hanging fruit.
Beta Was this translation helpful? Give feedback.
All reactions