Avoid using CBigNum for simple multiplications #3517

pull sipa wants to merge 1 commits into bitcoin:master from sipa:uintmult changing 4 files +34 −6
  1. sipa commented at 11:18 pm on January 11, 2014: member

    Implement a simple multiplication operator in uint256, and use it for the few cases where we used to convert to CBigNum for this.

    Tests are included that verify the behavior is identical to converting to CBigNum, doing the multiplication, and converting back.

    Overall goal: reduce dependency on OpenSSL where possible in core code.

  2. Avoid conversion to CBigNum for simple multiplications 87aacaa633
  3. BitcoinPullTester commented at 0:00 am on January 12, 2014: none
    Automatic sanity-testing: PASSED, see http://jenkins.bluematt.me/pull-tester/87aacaa633bfedaf9947e051dee47807bdeaff9b for binaries and test log. This test script verifies pulls every time they are updated. It, however, dies sometimes and fails to test properly. If you are waiting on a test, please check timestamps to verify that the test.log is moving at http://jenkins.bluematt.me/pull-tester/current/ Contact BlueMatt on freenode if something looks broken.
  4. laanwj commented at 8:43 am on January 12, 2014: member
    ACK
  5. sipa commented at 1:28 pm on April 21, 2014: member

    I completely forgot I wrote this.

    It’s superceded by #4076 now.

  6. sipa closed this on Apr 21, 2014

  7. DrahtBot locked this on Sep 8, 2021

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2024-11-17 06:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me