>> Take, for instance, bug 370. There the original bug fix was to change the >> algebra code from (something like) >> >> f(a+b) >> >> to >> >> c:=a+b >> f(c) >> >> When I went to apply this patch it didn't look right. But there was no >> documentation about what the patch was intended to do. It eventually turned >> out that the correct patch was to fix the spad compiler. > >Sorry, but do you really mean > >http://axiom-wiki.newsynthesis.org/370ImproperCodeEmmitedByMkUnionFunListImpactsDefintrfSpad > >? I cannot see any algebra code there - of course, possibly only the proper fix >(by Stephen Wilson) was sent to IssueTracker, but I'd like to make sure. > >Many thanks for clarification,
Yes, that's the bug. The original fix was in the algebra. I didn't think the original patch made sense and I started debugging the generated code, leading to the compiler. Stephen Wilson found and fixed the compiler bug. Tim _______________________________________________ Axiom-developer mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/axiom-developer
