yamt commented on code in PR #14923:
URL: https://github.com/apache/nuttx/pull/14923#discussion_r1857788934
##########
drivers/syslog/Kconfig:
##########
@@ -407,5 +407,25 @@ config SYSLOG_REGISTER
---help---
This option will support register the syslog channel
dynamically.
+config SYSLOG_STDOUT
Review Comment:
> Neither printf nor syslog can guarantee the integrity of printing, they
are have different protection mechanisms in the backend
implementation(txsem/csection)
what kind of "the integrity of printing" are you talking about?
after all, syslog and stdout are two separate streams.
i suspect you are expecting too much.
> The application is an open-source project. It is impossible to change all
stdout to syslog().
are you saying that my guess is correct? ie. it's a problem in your
application.
> This option is a configurable function, and default value is n, which
won't impact anyone.
it affects developers by complicating the configuration matrix.
--
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: commits-unsubscr...@nuttx.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
