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 2021/12/06 09:14:25 UTC

[GitHub] [incubator-nuttx] xiaoxiang781216 opened a new pull request #4944: libc/getopt: Move struct getopt_s to include/nuttx/tls.h

xiaoxiang781216 opened a new pull request #4944:
URL: https://github.com/apache/incubator-nuttx/pull/4944


   ## Summary
   and remove include/nuttx/lib/getopt.h
   
   ## Impact
   Code refactor only
   
   ## Testing
   Pass CI.
   


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

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] davids5 commented on pull request #4944: libc/getopt: Move struct getopt_s to include/nuttx/tls.h

Posted by GitBox <gi...@apache.org>.
davids5 commented on pull request #4944:
URL: https://github.com/apache/incubator-nuttx/pull/4944#issuecomment-986773656


   @xiaoxiang781216 Please list motivation for this change.


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

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] xiaoxiang781216 commented on pull request #4944: libc/getopt: Move struct getopt_s to include/nuttx/tls.h

Posted by GitBox <gi...@apache.org>.
xiaoxiang781216 commented on pull request #4944:
URL: https://github.com/apache/incubator-nuttx/pull/4944#issuecomment-986777106


   > @xiaoxiang781216 Please list motivation for this change.
   
   Two points here:
   1. Move all TLS related structure definition into one place
   2. Reduce NuttX specific header files


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

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [incubator-nuttx] davids5 merged pull request #4944: libc/getopt: Move struct getopt_s to include/nuttx/tls.h

Posted by GitBox <gi...@apache.org>.
davids5 merged pull request #4944:
URL: https://github.com/apache/incubator-nuttx/pull/4944


   


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

To unsubscribe, e-mail: commits-unsubscribe@nuttx.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org