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

Threshold: don't copy the entire vector just to cache the initial size. #10

Merged
merged 1 commit into from Sep 25, 2021

Conversation

furszy
Copy link

@furszy furszy commented Sep 5, 2021

auto-descriptive title.
The vector is being copied to a temp variable for no reason.

@furszy furszy self-assigned this Sep 5, 2021
Copy link

@Fuzzbawls Fuzzbawls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 46926a2

Copy link

@random-zebra random-zebra left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

utACK 46926a2

@furszy furszy merged commit bc64f12 into PIVX-Project:main Sep 25, 2021
furszy added a commit to furszy/bitcoin-core that referenced this pull request Sep 25, 2021
bc64f128f0 Merge PIVX-Project/bls-signatures#10: Threshold:  don't copy the entire vector just to cache the initial size.
8c75dd860d Merge PIVX-Project/bls-signatures#9: [Build] Fix autotools builds on M1 Mac CPU
46926a2216 Poly::Evaluate, don't copy the entire vector just to cache the initial vector size.
544d00ed23 [Build] Fix autotools builds on M1 Mac CPU

git-subtree-dir: src/chiabls
git-subtree-split: bc64f128f009fd8825f249090be9d64a74834093
furszy added a commit to PIVX-Project/PIVX that referenced this pull request Sep 27, 2021
ad308f2 Squashed 'src/chiabls/' changes from 676ea45e80..bc64f128f0 (furszy)

Pull request description:

  Updating the bls-sig repo to the latest version. Pulling:
  * PIVX-Project/bls-signatures#9
  * PIVX-Project/bls-signatures#10

ACKs for top commit:
  random-zebra:
    ACK ffc8e50

Tree-SHA512: 7cc8c9b9d267e9f2de1965cf17e055c724f441a9013b0557eb897fa4029d818008a3263ebe5be97a49ff4587480885caa9d101b92b24d629c6c3203f87a032bb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants