[riot-notifications] [RIOT-OS/RIOT] treewide: Address uncrustify suggestions (#16737)

Jan Romann notifications at github.com
Sat Aug 14 17:19:57 CEST 2021


@JKRhb commented on this pull request.



> @@ -305,7 +305,7 @@ typedef struct {
      * @param   clock       ztimer clock to set the new target
      * @param   val         Relative target (e.g. fire at value `now() + val`)
      */
-    void (*set)(ztimer_clock_t *clock, uint32_t val);
+    void (*set)(ztimer_clock_t * clock, uint32_t val);

Hmm, I am not sure, everywhere else the star is put before the parameter name (even in the case of the third function in this struct, see line 314). What is very odd is the fact that the star placement seems to only fail in the case of functions with return type `void`. If you change the return type from `void` to `uint32_t`, for example, the uncrustification works as expected 🤔

-- 
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/16737#discussion_r688942059
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.riot-os.org/pipermail/notifications/attachments/20210814/cb7b47cf/attachment.htm>


More information about the notifications mailing list