You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@mynewt.apache.org by GitBox <gi...@apache.org> on 2018/03/28 01:37:31 UTC

[GitHub] ccollins476ad opened a new pull request #969: Add convenience header: os/mynewt.h

ccollins476ad opened a new pull request #969: Add convenience header: os/mynewt.h
URL: https://github.com/apache/mynewt-core/pull/969
 
 
   This new header (`os/mynewt.h) includes the following headers:
   
       * syscfg/syscfg.h
       * sysinit/sysinit.h
       * sysflash/sysflash.h
       * os/os.h
       * defs/error.h
   
   This header is intended to be included before any other non-system headers.  The purpose of this header is to make an application developer's job easier, and simplify the "getting started" process.
   
   I think a good rule of thumb is: always include `os/mynewt.h` and never directly include any of the headers listed above.
   
   I put this header in the `kernel/os` package because, in practice, every other package has access to this one.  I am happy to move it somewhere else if there are alternative ideas.
   
   I was tempted to include `bsp/bsp.h` in this header, but I decided it is better if Mynewt doesn't make any assumptions about which files the BSP exports.  

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