Does anyone here have time to look at this? On 6/26/06, Alexander Belopolsky <[EMAIL PROTECTED]> wrote: > On 6/19/06, Guido van Rossum <[EMAIL PROTECTED]> wrote: > > On 6/14/06, Alexander Belopolsky <[EMAIL PROTECTED]> wrote: > > > ... It would be better to change the message > > > to "'Foo' object has only read-only attributes (assign to .bar)" as in > > > the case tp_setattro == tp_setattr == NULL in PyObject_SetAttr . > > > > I agree. Can you submit a patch to SF please? > > > Please see: > > https://sourceforge.net/tracker/index.php?func=detail&aid=1512942&group_id=5470&atid=305470 > > I've tested the patch by setting tp_setattr to 0 in Xxo_Type. With the patch: > > >>> import xx > >>> x = xx.new() > >>> x.a = 2 > Traceback (most recent call last): > File "<stdin>", line 1, in <module> > AttributeError: 'xxmodule.Xxo' object has only read-only attributes > (assign to .a) > >>> del x.a > Traceback (most recent call last): > File "<stdin>", line 1, in <module> > AttributeError: 'xxmodule.Xxo' object has only read-only attributes (del .a) > > Note that this log reveals a small inaccuracy in xxmodule.c : the > module name is "xx," but Xxo type name is "xxmodule.Xxo." Should I > submit a patch fixing that? >
-- --Guido van Rossum (home page: http://www.python.org/~guido/) _______________________________________________ Python-Dev mailing list Python-Dev@python.org http://mail.python.org/mailman/listinfo/python-dev Unsubscribe: http://mail.python.org/mailman/options/python-dev/archive%40mail-archive.com