Movatterモバイル変換


[0]ホーム

URL:


homepage

Issue28893

This issue trackerhas been migrated toGitHub, and is currentlyread-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title:Make sure exceptions raised in __aiter__ are properly chained in ceval
Type:behaviorStage:resolved
Components:Interpreter CoreVersions:Python 3.7, Python 3.6, Python 3.5
process
Status:closedResolution:fixed
Dependencies:Superseder:
Assigned To: yselivanovNosy List: brett.cannon, ned.deily, serhiy.storchaka, vstinner, yselivanov
Priority:normalKeywords:patch

Created on2016-12-07 04:41 byyselivanov, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.

Files
File nameUploadedDescriptionEdit
chain.patchyselivanov,2016-12-07 04:41review
Pull Requests
URLStatusLinkedEdit
PR 407mergedyselivanov,2017-03-02 23:07
PR 414mergedyselivanov,2017-03-03 03:29
PR 703larry,2017-03-17 21:00
PR 552closeddstufft,2017-03-31 16:36
Messages (6)
msg282595 -(view)Author: Yury Selivanov (yselivanov)*(Python committer)Date: 2016-12-07 04:41
Originally reported inhttp://bugs.python.org/issue28885#msg282538Ned, please take a look at the attached patch.  While I think that the change is quite small & safe to merge, I think it's OK if we push this only in 3.6.1.  It's a nice usability improvement, but only for people that implement objects with custom __await__, which is quite a rare case.
msg282597 -(view)Author: Serhiy Storchaka (serhiy.storchaka)*(Python committer)Date: 2016-12-07 05:44
Should the original exception be stored on the __context__ or the __cause__ attribute? In the latter case you should use _PyErr_FormatFromCause().
msg282659 -(view)Author: Ned Deily (ned.deily)*(Python committer)Date: 2016-12-07 20:06
Yury, sounds like a bug to me so it would be appropriate for a maintenance release.  And not a showstopper release critical.
msg282675 -(view)Author: Yury Selivanov (yselivanov)*(Python committer)Date: 2016-12-07 22:56
Ned, sounds good.Serhiy, we don't have _PyErr_FormatFromCause in 3.5, so I'll probably commit the patch as is in 3.5.  For 3.6 is definitely makes sense to use _PyErr_FormatFromCause.
msg288853 -(view)Author: Yury Selivanov (yselivanov)*(Python committer)Date: 2017-03-03 04:16
New changesetdea5101ae101aefed14de98e6bb1658f4cae8712 by Yury Selivanov in branch '3.6':bpo-28893: Set __cause__ for errors in async iteration protocol (#407)https://github.com/python/cpython/commit/dea5101ae101aefed14de98e6bb1658f4cae8712
msg290339 -(view)Author: Yury Selivanov (yselivanov)*(Python committer)Date: 2017-03-24 23:12
New changeset398ff91ac0b8f4d930cd5d9e3e6a4bf247f810ef by Yury Selivanov in branch 'master':bpo-28893: Set __cause__ for errors in async iteration protocol (#407)https://github.com/python/cpython/commit/398ff91ac0b8f4d930cd5d9e3e6a4bf247f810ef
History
DateUserActionArgs
2022-04-11 14:58:40adminsetgithub: 73079
2017-03-31 16:36:30dstufftsetpull_requests: +pull_request1030
2017-03-24 23:12:08yselivanovsetmessages: +msg290339
2017-03-17 21:00:36larrysetpull_requests: +pull_request624
2017-03-03 23:10:38yselivanovsetstatus: open -> closed
resolution: fixed
stage: patch review -> resolved
2017-03-03 04:16:06yselivanovsetmessages: +msg288853
2017-03-03 03:29:03yselivanovsetpull_requests: +pull_request345
2017-03-02 23:07:50yselivanovsetpull_requests: +pull_request339
2016-12-07 22:56:43yselivanovsetmessages: +msg282675
2016-12-07 20:06:02ned.deilysetmessages: +msg282659
2016-12-07 09:32:05serhiy.storchakalinkissue28885 dependencies
2016-12-07 05:44:46serhiy.storchakasetmessages: +msg282597
2016-12-07 04:41:33yselivanovcreate
Supported byThe Python Software Foundation,
Powered byRoundup
Copyright © 1990-2022,Python Software Foundation
Legal Statements

[8]ページ先頭

©2009-2026 Movatter.jp