Positive * pos_inf should be pos_inf, not 0 +- inf. Same for other combinations of signs.

flintlib / arb

Arb has been merged into FLINT -- use https://github.com/flintlib/flint/ instead

http://arblib.org/

GNU Lesser General Public License v2.1

457 stars 137 forks source link

Closed postmath closed 2 years ago

postmath commented 2 years ago

This addresses github issue #408.

fredrik-johansson commented 2 years ago

Thanks! Just two changes:

Move these branches below ARB_IS_LAGOM(...). These are fast paths for regular arithmetic and exclude infinities.
Add some test code (does not need to be randomized tests; you can just test the relevant special cases).

postmath commented 2 years ago

Thanks for the review.

fredrik-johansson commented 2 years ago

Looks great. Thanks!