Antoine Pitrou <pit...@free.fr> added the comment: > > 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. ---------- _______________________________________ Python tracker <rep...@bugs.python.org> <http://bugs.python.org/issue3243> _______________________________________ _______________________________________________ Python-bugs-list mailing list Unsubscribe: http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com