[riot-notifications] [RIOT-OS/RIOT] uncrustify: split lines at 100 chars (#10873)

Martine Lenders notifications at github.com
Sat Jan 26 13:16:25 CET 2019


Here is what I dread: If you want to make `uncrustify` the authoritative decision maker but with this change you basically break

> * Line length: aim for no more than 80 characters per line, the absolute maximum should be 100 characters per line.

from our coding conventions. I always interpreted this as 80 chars is desirable, but if you absolutely have to you can go up to hundred (e.g. with a break at 80 it would be less readable or URLs in doc [though those might actually overreach 100]). So I would say: set it to 80 and if there are instances where 100 is needed use `begin{code-style-ignore}` (unless there is a way to get `uncrustify` to interpret this as it is written...).

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/RIOT-OS/RIOT/pull/10873#issuecomment-457826648
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190126/f44b378c/attachment.html>


More information about the notifications mailing list