Hello all, The patch I sent last friday turned up in the source. That's the good part, but unfortunately at the wrong spot. The line const ex newarg = ex_to<pseries>((arg/coeff).series(rel, order+n, options)).shift_exponents(-n).convert_to_poly(true); should appear after the patch. If it comes before, the patch does not solve anything. Bye, Chris Dams
Hello. On Mon, Apr 19, 2004 at 10:24:43AM +0000, Chris Dams wrote:
Hello all,
The patch I sent last friday turned up in the source. That's the good part, but unfortunately at the wrong spot. The line
const ex newarg = ex_to<pseries>((arg/coeff).series(rel, order+n, options)).shift_exponents(-n).convert_to_poly(true);
should appear after the patch. If it comes before, the patch does not solve anything.
Oops, yes that happens if you apply patches manually ... ;-) But the applied patch is actually solving the problem, isn't it? So it's just a code style issue, or did I get it wrong? It will be corrected nevertheless. Bye, Jens
Hello Jens, On Mon, 19 Apr 2004, Jens Vollinga wrote:
Oops, yes that happens if you apply patches manually ... ;-) But the applied patch is actually solving the problem, isn't it? So it's just a code style issue, or did I get it wrong? It will be corrected nevertheless.
My original patch solved the issue, but the lines that are inserted in the patch appeared at the wrong spot, resulting again in non-termination of the series function. The line that I cited in my previous mail should definitely not appear before my patch because it causes non-termination. Bye, Chris Dams
participants (2)
-
Chris Dams
-
Jens Vollinga