HongChao6 commented on PR #3434:
URL: https://github.com/apache/nuttx-apps/pull/3434#issuecomment-4174200736

   > I suppose similarly, wouldn't you want to use fprintf to stderr instead of 
syslog errors? You can also remove the macros entirely and directly write 
printf since they're just wrappers.
   
   Good point, I'll switch GETEVENT_ERR to use fprintf(stderr, ...) instead of 
syslog.
   
   I'd prefer to keep the macros though — they provide a consistent [getevent]: 
prefix across all output, so if the format ever needs to change, it's a 
single-line edit. Without them, the prefix string would be duplicated in every 
call site.
   
   I'll update and push shortly.


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