Message122909
| Author |
xuanji |
| Recipients |
catlee, davide.rizzo, eric.araujo, jhylton, orsenthil, pitrou, rcoyner, rhettinger, xuanji |
| Date |
2010年11月30日.15:55:42 |
| SpamBayes Score |
2.823652e-09 |
| Marked as misclassified |
No |
| Message-id |
<1291132545.06.0.604337160417.issue3243@psf.upfronthosting.co.za> |
| In-reply-to |
| Content |
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) |
|