lzma implementation#5717
Conversation
|
This turned out surprisingly effective, about 2/3s of the tests pass, which is enough for me at the moment. The main missing parts involve a bunch of unsafe things that As a side note I also abstracted out |
Sorry, something went wrong.
c09442c to
2bad5d2
Compare
April 21, 2025 00:01
|
Lots of tests are failing because they were previously blocked due to no implementation at all. |
Sorry, something went wrong.
|
Would you be able to squash these commits down? Ideally into one for copying the files from cpython, and one for all the rest? |
Sorry, something went wrong.
|
Sure |
Sorry, something went wrong.
19a43f5 to
ca2dc44
Compare
April 21, 2025 19:40
|
It's at 3 now, I committed things in a weird fashion so I can't get it below that. |
Sorry, something went wrong.
|
oh no.. I thought #5728 and this one had shared work of the new compression module |
Sorry, something went wrong.
|
@arihant2math What will be the best way? Is it worth to keep #5728? |
Sorry, something went wrong.
|
Probably easiest to revert, yes. |
Sorry, something went wrong.
…tPython#5728)" This reverts commit 9c88475.
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
Signed-off-by: Ashwin Naren <arihant2math@gmail.com>
79646fd
into
RustPython:main
May 7, 2025
Based on #5709 because of the zlib abstraction.