Message122907
| Author |
pitrou |
| Recipients |
catlee, davide.rizzo, eric.araujo, jhylton, orsenthil, pitrou, rcoyner, rhettinger, xuanji |
| Date |
2010年11月30日.15:43:49 |
| SpamBayes Score |
0.023087896 |
| Marked as misclassified |
No |
| Message-id |
<1291131827.3538.17.camel@localhost.localdomain> |
| In-reply-to |
<1291131337.78.0.341827077103.issue3243@psf.upfronthosting.co.za> |
| Content |
> > len(data) will raise anyway.
>
> No, it won't, if the iterable happens to be a sequence.
Well, it seems the patch is confused between iterable and iterator. Only
iterators have a __next__, but they usually don't have a __len__.
The patch should really check for iterables, so it should use:
if isinstance(data, collections.Iterable)
raise ValueError#etc. |
|