A further remark:
The orginal code for genfact(a,b,c) was:
...
(cond ((and (fixnump b) (> b -1)) (gfact a b c))
((integerp b) (merror "Bad second argument to `genfact': ~:M" b))
...
Here only the second argument b was tested to be a positive fixnum. For a
general integer an error was thrown. Because of this restriction of b to a
fixnum I have restricted the arguments a and c to a fixnum too.
This can be changed.
Dieter Kaiser
-----Urspr?ngliche Nachricht-----
Von: maxima-bounces at math.utexas.edu [mailto:maxima-bounces at math.utexas.edu] Im
Auftrag von Robert Dodier
Gesendet: Montag, 22. Dezember 2008 17:20
An: Michael Abshoff; Maxima List
Cc: William Stein
Betreff: Re: [Maxima] Maxima 5.17.1 regressions for Sage
About a problem reported at: http://trac.sagemath.org/sage_trac/ticket/4841
"Bad argument to `genfact'"
What are the arguments to genfact here?
Inspecting the Maxima code, I see the following restriction:
(cond ((and (fixnump a)
(fixnump b)
(fixnump c))
(if (and (> a -1)
(> b -1)
(or (<= c a) (= b 0))
(<= b (/ a c)))
(gfact a b c)
(merror "Bad argument to `genfact'")))
(I.e. when a, b, and c are machine integers)
best
Robert Dodier
_______________________________________________
Maxima mailing list
Maxima at math.utexas.edu
http://www.math.utexas.edu/mailman/listinfo/maxima