linguini1 commented on code in PR #17018:
URL: https://github.com/apache/nuttx/pull/17018#discussion_r2399848396


##########
include/syslog.h:
##########
@@ -121,6 +121,10 @@
 #define LOG_INFO      6  /* Informational message */
 #define LOG_DEBUG     7  /* Debug-level message */
 
+#define LOG_LOWOUT    8  /* Log to console straightly

Review Comment:
   This will still conflict with the POSIX definitions per my earlier comment 
unfortunately. Maybe you could use a mask in the higher bits? Like (1 << 31)?
   
   This way the priority still can indicate the syslog level and subsystem, but 
the extra bit is a non-POSIX extension that allows you to log to the console 
directly for debugging.
   
   I'm still not sure if that's the best way to do it, because there are 
already existing macros for each subsystem to syslog that won't be able to log 
to your new sink unless they're modified to add that bit mask. It would only 
really help you if you're quickly debugging and you need to add in some 
temporary 'syslog' calls yourself that you remove later. That being said, I 
don't really have any better ideas. Maybe you could also try asking the mailing 
list.



-- 
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