Give minusInteger its own implementation.
Previously minusInteger used plusInteger and negateInteger, which meant it always allocated. Now it works more like plusInteger.
This looks great! Thanks @alan.
Just a few minor points,
- the name of the testcase is rather general given it is really just testing addition and subtraction
- the alignment isn't quite right as noted inline
That being said, these are pretty minor points
Alignment isn't quite right.
This looks all good to me, thanks!
I've commented one minor request inline, and another minor request would be to add a short entry to libraries/integer-gmp/changelog.md! :-)
|17 ↗||(On Diff #7766)|
Because I'm paranoid, I'd also add a few values slightly outside the Int range, and also around +/- maxBound :: Word