[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Terry J. Reedy added the comment: This is one of 4 duplicate issues, 3 with somewhat similar patches, with nearly disjoint nosy lists. Consolidating. -- nosy: +terry.reedy resolution: -> duplicate status: open -> closed superseder: -> Function calls taking a generator as star argumen

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Martin Panter added the comment: See also Issue 4806 -- nosy: +vadmium ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubs

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Changes by Antoine Pitrou : -- assignee: pitrou -> ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Georg Brandl added the comment: Is this ready to commit? -- assignee: -> pitrou nosy: +georg.brandl ___ Python tracker ___ ___ Python

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Changes by Guilherme Polo : Removed file: http://bugs.python.org/file13035/check_tpiter_before_overriding_msg.diff ___ Python tracker ___ ___

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Guilherme Polo added the comment: Thanks for taking a look, my bad for taking so long to reply. I'm adding a new patch now. This one, besides the fix needed mentioned above, disregards tp_iter from old class instances (since they are always available and tell us nothing). Added file: http://bu

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Antoine Pitrou added the comment: You must check tp_iter as well, not only Py_TPFLAGS_HAVE_ITER. Py_TPFLAGS_HAVE_ITER only tells you that the tp_iter field exists, not that it's non-NULL. (see the code for PyObject_GetIter() in Objects/abstract.c for an example) -- nosy: +pitrou __

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Guilherme Polo added the comment: (Btw, the suggestion to check for tp_iter came from Antoine Pitrou. It seemed fine to me.) ___ Python tracker ___ ___

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

Changes by Guilherme Polo : -- components: +Interpreter Core type: -> behavior versions: +Python 2.7, Python 3.1 ___ Python tracker ___ __

[issue5218] Check for tp_iter in ceval:ext_do_call before overriding exception message

New submission from Guilherme Polo : Hi, I find it weird that given this sample code: def g(): yield iter(None) list(*g()) I get this traceback: Traceback (most recent call last): File "a1.py", line 3, in list(*g()) TypeError: type object argument after * must be a sequence, not ge