[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Axiom-developer] history, was: Patch 3148, was: any?, member?, ...
From: |
Martin Rubey |
Subject: |
Re: [Axiom-developer] history, was: Patch 3148, was: any?, member?, ... |
Date: |
Wed, 23 Jun 2004 18:37:22 +0000 |
William Sit writes:
> I thought all occurrences of dummy in combfunc.spad were replaced. Besides
> the
> definition dummy=new()$Symbol, there were four occurrences in the functions
> product and summation, all were replaced effectively with inline calls to
> new()$Symbol assigned to a local variable. The dummy in the patch above is
> the
> only one left. So, the patch is correct as it is because dummy is now used
> only
> once in combfunc.spad. However, I think removing all references to dummy
> (including its definition) and using a local version each time will avoid
> possible future mistakes similar to the ones you are patching.
I agree.
> Regarding your suggestion to have a pamphlet file "that ... explains them", I
> think perhaps the pamphlet file should add another tag like "history" that
> could
> be extracted separately into a tex file documenting just the changes, but in
> chronological order. It will make those times when one wants to just read the
> final code easier, especially for files that needed many patches.
CVS does this automagically.
Martin
- [Axiom-developer] any?, member?, ..., Martin Rubey, 2004/06/23
- Re: [Axiom-developer] any?, member?, ..., William Sit, 2004/06/23
- Re: [Axiom-developer] any?, member?, ..., Martin Rubey, 2004/06/23
- Re: [Axiom-developer] any?, member?, ..., William Sit, 2004/06/23
- Re: [Axiom-developer] Patch 3148, was: any?, member?, ..., Martin Rubey, 2004/06/23
- Re: [Axiom-developer] Patch 3148, was: any?, member?, ..., William Sit, 2004/06/23
- Re: [Axiom-developer] history, was: Patch 3148, was: any?, member?, ...,
Martin Rubey <=