Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rust-themis: Allocate with try_reserve #1014

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Commits on Jun 27, 2023

  1. rust-themis: Impl From<TryReserveError> for Error

    It will allow us using ? with the try_* methods, like try_reserve.
    G1gg1L3s committed Jun 27, 2023
    Configuration menu
    Copy the full SHA
    b63e01c View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    33204a9 View commit details
    Browse the repository at this point in the history
  3. rust-themis: Run the skipped tests on the 32bit

    They were skipped because allocation on 32-bit platforms failed
    while trying to allocate something bigger than 2GB. This is due to
    usage of `.reserve` which panics if it couldn't fulfill the request.
    
    Since we traversed to the rust 1.58, we now have the `try_*` methods,
    including the `.try_reserve` which allows us handle panics gracefully.
    
    I've tested it manually on pi4 and it works!
    G1gg1L3s committed Jun 27, 2023
    Configuration menu
    Copy the full SHA
    1dee404 View commit details
    Browse the repository at this point in the history
  4. rust-themis: with_capacity -> try_reserve

    Because we don't have try_with_capacity or something similar.
    G1gg1L3s committed Jun 27, 2023
    Configuration menu
    Copy the full SHA
    24d4fd0 View commit details
    Browse the repository at this point in the history

Commits on Aug 8, 2023

  1. rust: Add a test for checking 4GB allocation

    It was intended only for 32 bit systems, but let's extend it to
    x64 as well.
    G1gg1L3s committed Aug 8, 2023
    Configuration menu
    Copy the full SHA
    a0b709f View commit details
    Browse the repository at this point in the history