| commit | 835cfdcb2ffdaa3066728f8268edb40f33f3b2e8 | |
| author | Kris Katterjohn <katterjohn@gmail.com> | |
| Sun, 15 Jul 2018 22:05:37 +0000 (15 17:05 -0500) | ||
| committer | Kris Katterjohn <katterjohn@gmail.com> | |
| Sun, 15 Jul 2018 22:05:37 +0000 (15 17:05 -0500) | ||
| tree | 6aa88e254f16a3194595d910513c39cf5baddd43 | treesnapshot (tar.gz zip) |
| parent | 11fc1a2fdf2d7defc089945763c6324310574bf9 | commitdiff |
Fix bug #3365: rat(1/3),modulus:6 gives incomprehensible error
This improves the error message in CRECIP and CRECIP-GENERAL when
we cannot take the inverse of the given integer with respect to a
given modulus.
I'm using the wording suggested by Stavros Macrakis in the bug
report. It's generally much clearer than the current message and
I can't think of any obvious improvements to it.
The new error message is (arguably) still not completely clear in
some cases. An example is calling rat(1/3) with modulus:3 because
CRECIP is given 0, and so the error message mentions 0 (not 3).
This improves the error message in CRECIP and CRECIP-GENERAL when
we cannot take the inverse of the given integer with respect to a
given modulus.
I'm using the wording suggested by Stavros Macrakis in the bug
report. It's generally much clearer than the current message and
I can't think of any obvious improvements to it.
The new error message is (arguably) still not completely clear in
some cases. An example is calling rat(1/3) with modulus:3 because
CRECIP is given 0, and so the error message mentions 0 (not 3).
| src/rat3a.lisp | diffblobblamehistory |