Well type=str is correct, so I'd prefer ValueError

On Saturday, May 9, 2015 at 1:46:42 PM UTC+2, Jeroen Demeyer wrote:
>
> On 2015-05-09 13:37, Vincent Delecroix wrote: 
> > Hello, 
> > 
> > I am trying to speed up the creation of polynomial rings. Doing so, I 
> > rewrote most of the factory PolynomialRing and I do have a question. 
> > What should we do with 
> > 
> >      sage: R.<x,y,z> = PolynomialRing(QQ, 'xyz') 
> > 
> > or 
> > 
> >      sage: R.<x,y,z> = PolynomialRing(QQ, 'abc') 
> > 
> > I see two reasonable options: 
> >   1 - raise a ValueError saying that the variable names are specified 
> twice 
> This should obviously be an error. But I guess a TypeError is more 
> applicable, that's what is normally raised when calling a function with 
> the wrong number or kind of arguments. 
>

-- 
You received this message because you are subscribed to the Google Groups 
"sage-devel" group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to sage-devel+unsubscr...@googlegroups.com.
To post to this group, send email to sage-devel@googlegroups.com.
Visit this group at http://groups.google.com/group/sage-devel.
For more options, visit https://groups.google.com/d/optout.

Reply via email to