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 2022/01/10 21:54:07 UTC

[GitHub] [incubator-nuttx] pkarashchenko commented on pull request #5201: inline: switch from inline to inline_function

pkarashchenko commented on pull request #5201:
URL: https://github.com/apache/incubator-nuttx/pull/5201#issuecomment-1009379846


   > How about applying this change only to the `static inline` functions from header files and simply removing the `inline` keyword from functions defined in source/implementation files?
   > 
   > Usually the compiler is smart enough to decide whether a function should be inlined and the `always_inline` attribute may induce an unintended increase on code size.
   
   Or as an alternative we can introduce `inline_function` and `forceinline_function` so that can be selectable in header and source code 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