search

Bluetooth-Devices / dbus-fast

A faster version of dbus-next
MIT License
35 stars 8 forks source link

feat: speed up unmarshalling by skipping unused unix_fds header #246

Closed bdraco closed 12 months ago

bdraco commented 12 months ago

We would always remove it so we should skip it instead

codecov[bot] commented 12 months ago

Codecov Report

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

Comparison is base (da30b04) 83.26% compared to head (2e5f2fa) 83.30%. Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #246 +/- ## ========================================== + Coverage 83.26% 83.30% +0.04% ========================================== Files 28 28 Lines 3256 3258 +2 Branches 676 677 +1 ========================================== + Hits 2711 2714 +3 + Misses 330 329 -1 Partials 215 215 ``` | [Files Changed](https://app.codecov.io/gh/Bluetooth-Devices/dbus-fast/pull/246?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Bluetooth-Devices) | Coverage Δ | | |---|---|---| | [src/dbus\_fast/\_private/unmarshaller.py](https://app.codecov.io/gh/Bluetooth-Devices/dbus-fast/pull/246?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Bluetooth-Devices#diff-c3JjL2RidXNfZmFzdC9fcHJpdmF0ZS91bm1hcnNoYWxsZXIucHk=) | `94.97% <100.00%> (+0.26%)` | :arrow_up: |

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.