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 2020/04/30 04:56:01 UTC

[GitHub] [incubator-nuttx-apps] btashton opened a new pull request #220: Fix uninitialized variable warning in ostest

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


   In lowpri_thread if sem_wait fail nwaiting will not be set even
   though the rest of the function uses it.  This is a failure
   so really just handling the compiler warning with this.
   
   Signed-off-by: Brennan Ashton <ba...@brennanashton.com>


----------------------------------------------------------------
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-apps] btashton commented on pull request #220: Fix uninitialized variable warning in ostest

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #220:
URL: https://github.com/apache/incubator-nuttx-apps/pull/220#issuecomment-621612550


   @patacongo  This really papers over the issue, but at this point the ostest has failed so I am not sure how much error handling to add around this vs just being explicit in the behaviour.


----------------------------------------------------------------
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-apps] btashton commented on pull request #220: Fix uninitialized variable warning in ostest

Posted by GitBox <gi...@apache.org>.
btashton commented on pull request #220:
URL: https://github.com/apache/incubator-nuttx-apps/pull/220#issuecomment-621618041


   A lot of unrelated style issues that I am resolving.


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