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 orsenthil
Recipients eric.araujo, jhylton, mcjeff, nadeem.vawda, orsenthil, pitrou
Date 2011-03-20.04:30:43
SpamBayes Score 1.5153606e-10
Marked as misclassified No
Message-id <20110320043036.GC3185@kevin>
In-reply-to <1300590193.07.0.81814695588.issue10050@psf.upfronthosting.co.za>
Content
Thanks again.

Agree to use of returning the block size to the returnhook callable.
But this is going be a deviation from the existing behavior which was
present to inform the user at regular intervals and did not bother to
provide any 'progressive size' related data, so this change should
documented as new behavior.

With respect to the docs patch in urllib2 howto I would do something
like this.

with open(local_filename) as fh: 
     content = fh.read()

Or *better* would demonstrate the filename argument passing and
reporthook functionality. If you wish, take this part alone, (the
howto document) update as separate patch, which should be committed
once this is in.

There is one comment which I forgot to mention earlier.

The current urlretrieve function is internally calling the URLOpener's
retrieve method. 

Those URLOpener class might need a DeprecationWarning while address
thing bug and that particular retrieve method can be updated to use
the updated facility just as bonus till the time it survives.

Antoine - any suggestions on the last point?
History
Date User Action Args
2011-03-20 04:30:44orsenthilsetrecipients: + orsenthil, jhylton, pitrou, nadeem.vawda, eric.araujo, mcjeff
2011-03-20 04:30:43orsenthillinkissue10050 messages
2011-03-20 04:30:43orsenthilcreate