[GitHub] andrzej-kaczmarek opened a new pull request #1528: kernel/os: Fix os_trace APIs

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

andrzej-kaczmarek opened a new pull request #1528: kernel/os: Fix os_trace APIs
URL: https://github.com/apache/mynewt-core/pull/1528
 
 
   os_trace_api.h uses local #defined symbols from .c file to determine
   whether traces should be enabled for given file or not. In order for
   this to work properly it has to be included only after such symbols
   are #defined.
   
   However, assert.h includes os/mynewt.h and the nasty side-effect is
   that we have os_everything included by including one of std library
   includes.
   
   Let's assert.h just include os_fault.h instead as this should not have
   any side-effects.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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