[issue16546] ast.YieldFrom needlessly has its expr value as optional

Changes by Jesús Cea Avión : -- nosy: +jcea ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.python.

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Changes by Mark Dickinson : -- resolution: -> fixed status: open -> closed ___ Python tracker ___ ___ Python-bugs-list mailing list U

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Roundup Robot added the comment: New changeset ffb2fa900cb7 by Mark Dickinson in branch '3.3': Issue #16546: make ast.YieldFrom argument mandatory. http://hg.python.org/cpython/rev/ffb2fa900cb7 New changeset 9bc6e157475a by Mark Dickinson in branch 'default': Issue #16546: merge fix from 3.3 htt

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Mark Dickinson added the comment: New patch: move new test to the correct test class. -- Added file: http://bugs.python.org/file28103/issue16546_v2.patch ___ Python tracker ___ _

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Changes by Mark Dickinson : Removed file: http://bugs.python.org/file28094/issue16546.patch ___ Python tracker ___ ___ Python-bugs-list mailin

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Mark Dickinson added the comment: New patch that removes some unnecessary braces in the case statements. -- Added file: http://bugs.python.org/file28095/issue16546.patch ___ Python tracker _

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Mark Dickinson added the comment: Here's a patch. -- keywords: +patch Added file: http://bugs.python.org/file28094/issue16546.patch ___ Python tracker ___ ___

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Changes by Mark Dickinson : -- nosy: +benjamin.peterson ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Mark Dickinson added the comment: Ah, okay. I only looked at Grammar/Grammar, which looks fine. The doc issue is that at http://docs.python.org/dev/library/ast.html, under the heading "31.2.2. Abstract Grammar", it says: | Yield(expr? value) | YieldFrom(expr? value) But the do

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Brett Cannon added the comment: Where is the doc issue? And it's a bit more than just a doc issue anyway as Python.asdl has the same issue and that is what is used to generate the code that represents the nodes. -- ___ Python tracker

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Mark Dickinson added the comment: Brett: can you clarify---in what sense is the argument optional? I see that there's a doc issue at http://docs.python.org/dev/library/ast.html, where it's marked as optional but shouldn't but. Is there something more that's wrong? If you mean that ast.YieldF

[issue16546] ast.YieldFrom needlessly has its expr value as optional

Changes by Daniel Urban : -- nosy: +daniel.urban ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: http://mail.py

[issue16546] ast.YieldFrom needlessly has its expr value as optional

New submission from Brett Cannon: The grammar guarantees that 'yield from' statements have an expression to evaluate, plus a 'yield from' without an expression makes no sense. -- components: Library (Lib) messages: 176261 nosy: brett.cannon priority: normal severity: normal stage: test