oldk1331 wrote: > > along with some improvements: > > 1. I changed local type abbreviation to avoid conflict: > I changed "UP ==> SparseUnivariatePolynomial R" > into "SUPR ==> SparseUnivariatePolynomial R", > so that a spad expression "x::SUPR" can be copy & pasted > into interpreter, easier for debugging. > > 2. I changed the reduce syntax "op/[list]" into "reduce(op,[list])" > for the same reason. > > Do you agree with these 2 style changes and agree that > these changes can be applied to other spad files?
The two do not look like good changes to me. In both cases original code is more readable than code after change, so there is a loss. There is some gain for debugging, but before I can pass expression to interpreter in general I need to set up context. I can adjust names as part of creating context. So, IMO gain is very small. -- Waldek Hebisch -- You received this message because you are subscribed to the Google Groups "FriCAS - computer algebra system" group. To unsubscribe from this group and stop receiving emails from it, send an email to fricas-devel+unsubscr...@googlegroups.com. To post to this group, send email to fricas-devel@googlegroups.com. Visit this group at https://groups.google.com/group/fricas-devel. For more options, visit https://groups.google.com/d/optout.