MatthieuDartiailh / bytecode

Python module to modify bytecode
https://bytecode.readthedocs.io/
MIT License
302 stars 38 forks source link

Python 3.12 support #122

Closed MatthieuDartiailh closed 1 year ago

codecov-commenter commented 1 year ago

Codecov Report

Patch coverage: 96.90% and project coverage change: +0.22% :tada:

Comparison is base (987b00e) 95.37% compared to head (98e995d) 95.60%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #122 +/- ## ========================================== + Coverage 95.37% 95.60% +0.22% ========================================== Files 6 6 Lines 1925 2000 +75 Branches 456 481 +25 ========================================== + Hits 1836 1912 +76 + Misses 55 53 -2 - Partials 34 35 +1 ``` | [Files Changed](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh) | Coverage Δ | | |---|---|---| | [src/bytecode/\_\_init\_\_.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL19faW5pdF9fLnB5) | `89.92% <ø> (ø)` | | | [src/bytecode/bytecode.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL2J5dGVjb2RlLnB5) | `96.33% <0.00%> (ø)` | | | [src/bytecode/concrete.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL2NvbmNyZXRlLnB5) | `96.66% <96.29%> (+0.08%)` | :arrow_up: | | [src/bytecode/instr.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL2luc3RyLnB5) | `97.68% <98.48%> (+0.70%)` | :arrow_up: | | [src/bytecode/cfg.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL2NmZy5weQ==) | `93.44% <100.00%> (ø)` | | | [src/bytecode/flags.py](https://app.codecov.io/gh/MatthieuDartiailh/bytecode/pull/122?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=Matthieu+Dartiailh#diff-c3JjL2J5dGVjb2RlL2ZsYWdzLnB5) | `98.52% <100.00%> (+0.02%)` | :arrow_up: |

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

MatthieuDartiailh commented 1 year ago

Just a quick update:

MatthieuDartiailh commented 1 year ago

Framework tests have detected an issue with the computation of jumps.

It appears to be related to the computation of jump targets when the jumping instruction make use of an extended arg...

MatthieuDartiailh commented 1 year ago

A single test in the framework test and does not look bytecode related.

MatthieuDartiailh commented 1 year ago

This is pending some documentation update before it can be merged.