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
-~----------~----~----~----~------~----~------~--~---

Reply via email to