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/02/29 05:48:34 UTC

[GitHub] [incubator-nuttx-apps] masayuki2009 opened a new pull request #112: testing: ostest: Fix a warning in cond.c for non-SMP configurations.

masayuki2009 opened a new pull request #112: testing: ostest: Fix a warning in cond.c for non-SMP configurations.
URL: https://github.com/apache/incubator-nuttx-apps/pull/112
 
 
   ### Summary
   
   - This PR will fix a warning in cond.c for non-SMP configurations.
   
   cond.c: In function 'thread_signaler':
   cond.c:184:5: warning: "CONFIG_SMP_NCPUS" is not defined, evaluates to 0 [-Wundef]
     184 | #if CONFIG_SMP_NCPUS > 1
         |     ^~~~~~~~~~~~~~~~
   
   ### Impact
   
   - Only ostest (actually cond_test) will be affected.
   
   ### Testing
   
   - I tested this PR with both  spresense:smp and spresense:wifi (non-SMP).
   

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

[GitHub] [incubator-nuttx-apps] xiaoxiang781216 merged pull request #112: testing: ostest: Fix a warning in cond.c for non-SMP configurations.

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 merged pull request #112: testing: ostest: Fix a warning in cond.c for non-SMP configurations.
URL: https://github.com/apache/incubator-nuttx-apps/pull/112
 
 
   

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