Added comment explaining sigusr1_handler function#2223
Added comment explaining sigusr1_handler function#2223HarshithaNimmala wants to merge 9 commits intoCCExtractor:masterfrom
Conversation
CCExtractor CI platform finished running the test files on linux. Below is a summary of the test results, when compared to test for commit 03ad9e8...:
Your PR breaks these cases:
NOTE: The following tests have been failing on the master branch as well as the PR:
Congratulations: Merging this PR would fix the following tests:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
CCExtractor CI platform finished running the test files on windows. Below is a summary of the test results, when compared to test for commit 03ad9e8...:
Your PR breaks these cases:
NOTE: The following tests have been failing on the master branch as well as the PR:
Congratulations: Merging this PR would fix the following tests:
It seems that not all tests were passed completely. This is an indication that the output of some files is not as expected (but might be according to you). Check the result page for more info. |
|
Closing — adding a single comment to a self-explanatory signal handler is not a useful contribution. Please focus on substantive changes. |
Summary
Added a comment explaining the purpose of the sigusr1_handler function.
Motivation
Improves code readability and helps new contributors understand the signal handling logic.
Changes
Added descriptive comment above sigusr1_handler
Impact
No functional changes. Improves documentation within the code.