xiaoxiang781216 commented on code in PR #12652:
URL: https://github.com/apache/nuttx/pull/12652#discussion_r1670783619


##########
drivers/note/note_initialize.c:
##########
@@ -52,11 +52,11 @@
 
 int note_early_initialize(void)
 {
-  int ret = 0;
+  int ret = OK;

Review Comment:
   but why relace 0 with OK? if possible, it's better to remove OK and ERROR 
from include/sys/types.h since the symbol confliction happen many time because 
include/sys/types.h has this non-standard but very common symbol.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to