You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/03/28 20:27:24 UTC

[GitHub] [incubator-nuttx] btashton opened a new pull request #3215: syslog: Drop extra carriage return from syslog calls

btashton opened a new pull request #3215:
URL: https://github.com/apache/incubator-nuttx/pull/3215


   ## Summary
   `\r`  was explicitly added to a lot of syslog calls. The use of a carriage return is controlled by the driver itself.
   
   NOTE: While making this change I saw a few cases where we are calling syslog directly or even worse calling printf.  I will handle this in another PR later.
   
   ## Impact
   Carriage return is controlled by syslog driver as expected.
   
   ## Testing
   Minimal, but this change is fixing the call convention.
   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3215: syslog: Drop extra carriage return from syslog calls

Posted by GitBox <gi...@apache.org>.

xiaoxiang781216 merged pull request #3215:
URL: https://github.com/apache/incubator-nuttx/pull/3215


   


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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org