xiaoliud opened a new pull request, #4034:
URL: https://github.com/apache/nuttx-apps/pull/4034

   ## Summary
   
   Register the existing termination handler for SIGTERM as well as SIGINT,
   so kill <pid> stops the event loop and getevent closes its devices
   before exiting. Before this change, kill -15 left getevent running.
   
   The first commit only fixes nxstyle issues in getevent.c. 
   
   ## Impact
   
   - Only graphics/input/getevent.
   - SIGINT handling is unchanged.
   - No Kconfig, API, or documentation changes.
   
   ## Testing
   
   - Host: Ubuntu 22.04 x86_64, gcc 13.1.0; sim:nsh with
     CONFIG_GRAPHICS_INPUT_GETEVENT=y.
   - In the tested sim configuration, getevent remained listed in `ps`
   after `kill -15 <pid>` before this change. After this change, it
   prints "Terminating!" and is no longer listed in `ps`.
   `kill -2 <pid>` also continues to terminate getevent normally.
   - checkpatch.sh -c -u -m -g on the range and on each commit: pass.
   - No hardware testing performed.
   
   
   


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to