On May 14, 2009, at 1:27 PM, Craig Citro wrote: > >> This is the right fix. Looks good--are you sure we don't use pari's >> sum anywhere else? >> > > Well, I'm not 100% sure ... but given that the Python and pari ones > accept *different* numbers of arguments, I suspect we're okay. I tried > using search_src to find cases with two or more commas in a call to > sum, but didn't see anything suspicious. (My regexp-fu is also not the > strongest, so I can't be sure I found everything.)
Good point, something trying to use 3 arguments would have failed at the compilation stage. > If we run into anything in the alpha0 cycle, I'll make sure to take > care of it ... I second the positive review (looks like Michael beat me to it). - Robert --~--~---------~--~----~------------~-------~--~----~ To post to this group, send email to sage-devel@googlegroups.com To unsubscribe from this group, send email to sage-devel-unsubscr...@googlegroups.com For more options, visit this group at http://groups.google.com/group/sage-devel URLs: http://www.sagemath.org -~----------~----~----~----~------~----~------~--~---