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.

Sign up for GitHub

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

Jump to bottom

Remove unnecessary cloning #62

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

Remove unnecessary cloning #62

wants to merge 5 commits into from

Commits on Apr 5, 2024

  1. point_to_vec takes reference, returns ref. Avoids cloning

    @mmagician
    mmagician committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    b7bcd07 View commit details
    Browse the repository at this point in the history

  2. can skip cloning the column during verification 

    column (&Vec) implements borrow
    @mmagician
    mmagician committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    c7abfd7 View commit details
    Browse the repository at this point in the history

  3. also no need to clone leaves for checking merkle paths, consume directly

    @mmagician
    mmagician committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    3838223 View commit details
    Browse the repository at this point in the history

  4. remove unnecessary clone of hash params

    @mmagician
    mmagician committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    06f574c View commit details
    Browse the repository at this point in the history

  5. fmt

    @mmagician
    mmagician committed Apr 5, 2024
    Configuration menu
    Copy the full SHA
    0c1e6c3 View commit details
    Browse the repository at this point in the history