[riot-notifications] [RIOT-OS/RIOT] uncrustify: change en/disable markers to "*auto-format-[on|off]*" (#10865)

Martine Lenders notifications at github.com
Fri Jan 25 12:37:17 CET 2019


miri64 commented on this pull request.

I'm aware this is very much a question of taste, but

1. why keep the asterisks? 
2. why not a more environment-like thing. It feels unintuitive to me that to enter a certain region of code you say "off" and to leave it you say "on". I'd rather expect something like
   ```C
   /* begin{code-style-ignore} */
   some* dirty=code;
   /* end{code-style-ignore} */
   ```

   (totally inspired by Latex ;P). 



-- 
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/10865#pullrequestreview-196468139
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190125/c7bcd355/attachment.html>


More information about the notifications mailing list