uncrustify-riot.cfg: update to coding convention#16149
uncrustify-riot.cfg: update to coding convention#16149kaspar030 merged 1 commit intoRIOT-OS:masterfrom
Conversation
The coding convention was changed to suggest breaking lines with with more than 100 chars. This brings the uncrustify configuration back in sync.
|
I guess we should apply the config to the uncrustify whitelist? |
|
If the files were already uncrustified with 80 char lines, there should be no change if we bump the max allowed length to 100. |
I read this as: When run locally, |
no, as ben understood it, I suggested re-applying the new config to everything whitelisted for uncrustify. |
|
I'm almost certain that uncrustify doesn't touch line breaks unless lines are over the char limit. I often break lines when uncrustufy demands it, but in more readable way. So far, uncrustify always accepted my alternative line breaking. From that, it seems most plausible that uncrustify is pretty trivial and really only insert line breaks when lines are too long, but never removes them. |
|
Is this still waiting for something? |
|
ACK. |
|
Thanks :-) |
Contribution description
The coding convention was changed to suggest breaking lines with with
more than 100 chars. This brings the uncrustify configuration back in
sync.
Testing procedure
I don't think this needs testing.
Issues/PRs references
Change of coding convention was done in #15793