Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: indygreg/python-zstandard
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: main
Choose a base ref
...
head repository: indygreg/python-zstandard
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: next
Choose a head ref
Checking mergeability… Don’t worry, you can still create the pull request.
  • 6 commits
  • 14 files changed
  • 1 contributor

Commits on Feb 16, 2021

  1. rust: create semi-safe functionis for constructing [CD]Dict 

    It isn't completely safe due to missing lifetime annotation. But at least
the unsafe is isolated to zstd_safe.
    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    ffc34da View commit details
    Browse the repository at this point in the history

  2. ci: regenerate requirements.txt with latest pip-tools 

    The syntax changed a bit.
    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    ba615ad View commit details
    Browse the repository at this point in the history

  3. ci: update packages to latest

    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    c7ddd5c View commit details
    Browse the repository at this point in the history

  4. rust: implement buffer types 

    All tests pass.
    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    0f8be66 View commit details
    Browse the repository at this point in the history

  5. rust: implement ZstdCompressor.multi_compress_to_buffer() 

    The implementation is horribly inefficient compared to the C version
due to excessive memory copying. But it should be functionally correct
and it is multi-threaded, courtesy of the rayon crate.
    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    cb8f6e4 View commit details
    Browse the repository at this point in the history

  6. rust: implement ZstdDecompressor.multi_decompress_to_buffer() 

    Like the compressor implementation, this code isn't optimal due to
excessive buffer copies.

With this change, all tests pass with the Rust backend!
    @indygreg
    indygreg committed Feb 16, 2021
    Configuration menu
    Copy the full SHA
    6bb85c3 View commit details
    Browse the repository at this point in the history
Loading