search

dartist / dart-bignum

Other
14 stars 11 forks source link

Integrate native big integer #33

Closed rinick closed 9 years ago

rinick commented 9 years ago

This change make bignum lib use the native big integer when running in dartvm, because it needs some new int functions introduced in dart 1.11, it still fallback to old js mode when running in old sdk where those new functions are not supported.

for the js big integer, I also moved some initialization code to static function, since they don't need to be run for every BitInteger instace.

this change passed all the unit tests as well as all unit tests in the cipher pub library.

adam-singer commented 9 years ago

@rinick @kaendfinger I don't have time to maintain this code base at the moment. I can add you both to this repository and as uploaded to the pub site. Ping me over email to let me know which email addresses should be added. Thanks!

adam-singer commented 9 years ago

@rinick @kaendfinger k found your emails in my contacts list, both added, will merge pr.

sethladd commented 9 years ago

Thanks Adam. On Sun, Jun 28, 2015 at 08:39 Adam Singer notifications@github.com wrote:

Merged #33 https://github.com/dartist/dart-bignum/pull/33.

— Reply to this email directly or view it on GitHub https://github.com/dartist/dart-bignum/pull/33#event-342197882.