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 2020/08/30 05:46:15 UTC

[GitHub] [incubator-nuttx] sebastianene07 commented on pull request #1663: Refine the signal handling in sim

sebastianene07 commented on pull request #1663:
URL: https://github.com/apache/incubator-nuttx/pull/1663#issuecomment-683379630


   > @sebastianene07 please review this patch. To make #1602 enter the mainline, I would suggest you don't put all code into one patch, it's better to split that patch into two:
   > 1.Replace setjmp/longjmp with ucontext API
   > 2.Implement preemptable oneshot timer
   
   I think this is a good idea to split the patch it makes the review  more efficient. Thanks !


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

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