latchset / tang

Tang binding daemon
GNU General Public License v3.0
468 stars 56 forks source link

use sigaction instead of signal for SIGCHLD processing #110

Closed phirince closed 1 year ago

phirince commented 1 year ago

The issue being handled is explained in this issue: #109

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.24 :tada:

Comparison is base (7c0d86f) 55.88% compared to head (d5ee7b0) 56.12%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #110 +/- ## ========================================== + Coverage 55.88% 56.12% +0.24% ========================================== Files 4 4 Lines 544 547 +3 Branches 145 145 ========================================== + Hits 304 307 +3 Misses 140 140 Partials 100 100 ``` | [Impacted Files](https://app.codecov.io/gh/latchset/tang/pull/110?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=latchset) | Coverage Δ | | |---|---|---| | [src/socket.c](https://app.codecov.io/gh/latchset/tang/pull/110?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=latchset#diff-c3JjL3NvY2tldC5j) | `51.78% <100.00%> (+1.32%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

sarroutbi commented 1 year ago

@sergio-correia: can you please double-check this?