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/29 11:58:23 UTC

[GitHub] sdalu opened a new issue #972: os: moving os/queue.h to sys/queue.h

sdalu opened a new issue #972: os: moving os/queue.h to sys/queue.h 
URL: https://github.com/apache/mynewt-core/issues/972
 
 
   Moving os/queue.h to it's expected place: sys/queue.h (as it is a pretty standard include)
   So that's possible to write portable code across OS (otherwise there will be a clash when trying to import our own copy of sys/queue.h)
   
   Possibilities are (in my order of preference):
   * effectively moving file to `sys/queue.h` and creating an `os/queue.h` (for compatibilty with previous mynewt code)
   * creating a `sys/queue.h` wrapper: 
   ~~~
   #ifnded _SYS_QUEUE_H_
   #define _SYS_QUEUE_H_
   #include "os/queue.h"
   #endif
   ~~~
   * replacing define with `_SYS_QUEUE_H_` instead of  `_QUEUE_H_`. This will avoid inclusion of developper own copy of `sys/queue.h`, but a `sys/queue.h` will need to be provided
   

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