[riot-notifications] [RIOT-OS/RIOT] uncrustify: change en/disable markers (#10865)

Martine Lenders notifications at github.com
Fri Jan 25 18:30:50 CET 2019


miri64 approved this pull request.

Ran it on an Arch machine again. Works like a charm. As a general rule we should however add reasonings to these blocks as you did in #10867.

> @@ -96,7 +96,7 @@ void board_init_clock(void)
      /* Note: The range is limited to ~100MHz and depends on PLL settings */
     PRCI_set_hfrosctrim_for_f_cpu(CPU_DESIRED_FREQ, PRCI_FREQ_UNDERSHOOT);
 
-     /* disable uncrustify *INDENT-OFF* */
+     /* begin{code-style-ignore} */

Uncrustify moves this comment one space to the left. Why not fix it ;-)? (the code below probably needs to be moved as well though :-/)

-- 
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-196614427
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20190125/55a80d6f/attachment-0001.html>


More information about the notifications mailing list