You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The code itself is complex, which is OK to obtain a small footprint. It would help understand the code if the style was consistent. Right now, I see for example a mixture of:
if (...)
{
do_something();
if (...)
{ do_something();
These days, I would recommend trying to follow the Linux kernel coding style. Not that I like it more than others, but it does result in compact and very readable code, and it is widely used. Other widely followed coding styles include the GTK coding style and GNU coding standards.
Current style is OK too (3 space for indents for example) but it needs to be consistent and consistently applied.
The text was updated successfully, but these errors were encountered:
I agree that there should be some more consistency, though I'd prefer to enact more consistency in the least invasive way so that the line attribution is still useful in most cases. Whitespace-only changes are fine because git has an option to ignore them when obtaining line attribution, but I'm not sure how it handles e.g. moving from the second snippet to the first snippet in your example, I'll have to check that.
I have a hard time reading the code.
The code itself is complex, which is OK to obtain a small footprint. It would help understand the code if the style was consistent. Right now, I see for example a mixture of:
These days, I would recommend trying to follow the Linux kernel coding style. Not that I like it more than others, but it does result in compact and very readable code, and it is widely used. Other widely followed coding styles include the GTK coding style and GNU coding standards.
Current style is OK too (3 space for indents for example) but it needs to be consistent and consistently applied.
The text was updated successfully, but these errors were encountered: