Re: [Axiom-developer] history, was: Patch 3148, was: any?, member?, ...

[Martin Rubey]

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