This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author pitrou
Recipients jcon, nadeem.vawda, neologix, petri.lehtinen, pitrou, rich-noir
Date 2011-06-06.14:56:05
SpamBayes Score 1.96515e-07
Marked as misclassified No
Message-id <1307372162.3506.5.camel@localhost.localdomain>
In-reply-to <>
> If being pretty is the only reason for this choice, then I think that
> documenting the method as
> method:: read([n])
> is simpler and cleaner .
> But you've got much more experience than me, so I won't argue any further :-)

There are contexts where it is easier to give the default argument than
call the method without argument, though, and that's where I find None
more intuitive than -1 :) Arguably it's not very important, though.

> > Can I use _PyIO_ConvertSsize_t? Or should I duplicate its
> > functionality in mmapmodule.c?
> I let Antoine answer that.

I'm not sure. This would require including iomodule.h, which is supposed
to be private to the _io module. I think duplicating it is fine, since
the code is probably simple anyway (I'm too lazy to take a look right
now :-)).
Date User Action Args
2011-06-06 14:56:06pitrousetrecipients: + pitrou, nadeem.vawda, neologix, jcon, rich-noir, petri.lehtinen
2011-06-06 14:56:05pitroulinkissue12021 messages
2011-06-06 14:56:05pitroucreate