Skip to content

Conversation

@Warrows
Copy link

@Warrows Warrows commented Oct 11, 2018

Closes #717

@ghost ghost assigned Warrows Oct 11, 2018
@ghost ghost added the review label Oct 11, 2018
Copy link
Collaborator

@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.

Concept ACK

However, some styling changes are needed after doing an initial once-over look.

@Warrows Warrows force-pushed the zerocoin_bignum_refactor branch from 9d4c1d1 to 32dd307 Compare October 12, 2018 10:47
@Warrows
Copy link
Author

Warrows commented Oct 12, 2018

Styling changes done.

@Warrows Warrows added this to the 3.2.0 milestone Nov 16, 2018
@Warrows Warrows force-pushed the zerocoin_bignum_refactor branch from 32dd307 to 80ea9dd Compare December 26, 2018 12:18
@Fuzzbawls Fuzzbawls modified the milestones: 3.2.0, Future Jan 6, 2019
@furszy
Copy link

furszy commented Mar 5, 2019

Concept ACK.

@Fuzzbawls
Copy link
Collaborator

needs rebase

@Warrows Warrows force-pushed the zerocoin_bignum_refactor branch 2 times, most recently from 0906ebf to d91ad90 Compare April 1, 2019 10:59
@Warrows Warrows force-pushed the zerocoin_bignum_refactor branch from d91ad90 to 4e4975b Compare April 1, 2019 11:00
@Warrows
Copy link
Author

Warrows commented Apr 1, 2019

Rebased.

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.

ACK 4e4975b

Copy link
Collaborator

@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.

ACK 4e4975b

@furszy
Copy link

furszy commented Apr 29, 2019

ACK 4e4975b

@Fuzzbawls Fuzzbawls merged commit 4e4975b into PIVX-Project:master Apr 29, 2019
Fuzzbawls added a commit that referenced this pull request Apr 29, 2019
4e4975b [Refactor] Refactor bignum header file into several files (warrows)

Tree-SHA512: 52f0ad549bfd03731ac388ad8f07ae59d960549f49ac1f73c228ce14b1ef227f34408efce6198997754a230f8fae879939cbcfb75ffc16045a774849129242b4
@ghost ghost removed the review label Apr 29, 2019
@Warrows Warrows deleted the zerocoin_bignum_refactor branch April 29, 2019 09:05
@Fuzzbawls Fuzzbawls modified the milestones: Future, 3.3.0 May 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor bignum.h

4 participants