[GitHub] [incubator-nuttx] davids5 commented on a change in pull request #31: Documentation/NuttXCCodingStandard.html: Remove requirement to decor…

[GitBox <gi...@apache.org>]

davids5 commented on a change in pull request #31: Documentation/NuttXCCodingStandard.html:  Remove requirement to decor…
URL: https://github.com/apache/incubator-nuttx/pull/31#discussion_r362895441
 
 

 ##########
 File path: Documentation/NuttXCCodingStandard.html
 ##########
 @@ -2182,9 +2182,8 @@ <h2><a name="retvalues">Returned Values</a></h2>
   <b>Checking Return Values</b>.
   Callers of internal OS functions should always check return values for an error.
   At a minimum, a debug statement should indicate that an error has occurred.
-  The calling logic intentionally ignores the returned value, then the function return value should be explicitly cast to <code>(void)</code> to indicate that the return value is intentionally ignored.
 
 Review comment:
   Why the relaxation? What was the argument for these changes?

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


With regards,
Apache Git Services