xiaoxiang781216 commented on PR #6197:
URL: https://github.com/apache/incubator-nuttx/pull/6197#issuecomment-1133599529

   > Thanks for the comments, I'll try to tidy up the exit() functions a bit 
better. My idea was to do this temporary patch first and then clean everything 
up in the next PR, but I can try to do this PR better.
   > 
   > > But why not enable the compile of lib_exit?
   > 
   > Well I can of course do that, just need to omit the call to the exit 
functions. So if this is OK to you I can do it like this:
   > 
   > ```
   > void exit(int status)
   > {
   > #if 0 /* REVISIT: Enable this when kernel side atexit/on_exit is removed */
   >   atexit_call_exitfuncs(status);
   > #endif
   >   /* Call nx_exit() which will do the rest of the clean up and exit */
   > 
   >   nx_exit(status);
   > }
   > ```
   
   How about create a new patch in this PR to remove the exit callback from 
kernel side? And remove the kernel atexit implementation, and enable libc's 
version.


-- 
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-unsubscr...@nuttx.apache.org

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

Reply via email to