Message122909
davide: yeah, hasattr("lol", '__next__') == False, even though strings are Iterable; so for strings and other such sequences the len(data) line will be executed. So technically we shouldn't say "No Content-Length specified for iterable body" but we should say "No Content-Length specified for iterable body that is not a sequence".
Basically, this whole patch (both parts of it) will be much better off iif there is a clean way to say "a is an iterable but a is not a sequence", because even though b'this is a message' is Iterable, we want to treat it differently compared to, say, a generator object; we do NOT want to use the Iterator features (iter, next) of it, we want to use the sequencey features (by sending the whole chunk of it, by calling len) |
|
Date |
User |
Action |
Args |
2010-11-30 15:55:45 | xuanji | set | recipients:
+ xuanji, jhylton, rhettinger, orsenthil, pitrou, catlee, eric.araujo, rcoyner, davide.rizzo |
2010-11-30 15:55:45 | xuanji | set | messageid: <1291132545.06.0.604337160417.issue3243@psf.upfronthosting.co.za> |
2010-11-30 15:55:42 | xuanji | link | issue3243 messages |
2010-11-30 15:55:42 | xuanji | create | |
|