Movatterモバイル変換


[0]ホーム

URL:


homepage

Issue27911

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:Unnecessary error checks in exec_builtin_or_dynamic
Type:enhancementStage:resolved
Components:Interpreter CoreVersions:Python 3.6, Python 3.5
process
Status:closedResolution:fixed
Dependencies:Superseder:
Assigned To: brett.cannonNosy List: brett.cannon, eric.snow, ncoghlan, python-dev, xiang.zhang
Priority:normalKeywords:patch

Created on2016-08-31 11:22 byxiang.zhang, last changed2022-04-11 14:58 byadmin. This issue is nowclosed.

Files
File nameUploadedDescriptionEdit
exec_built_or_dynamic.patchxiang.zhang,2016-08-31 11:22review
Messages (5)
msg274015 -(view)Author: Xiang Zhang (xiang.zhang)*(Python committer)Date: 2016-08-31 11:22
The two PyErr_Occurred check in exec_builtin_or_dynamic are unnecessary. The PyModule_Check at the beginning of exec_builtin_or_dynamic has eliminated the possibility for PyModule_GetDef/State to fail.
msg274043 -(view)Author: Brett Cannon (brett.cannon)*(Python committer)Date: 2016-08-31 17:25
Patch LGTM. Thanks for catching that, Xiang! I'll commit your fix when I can (might not be until after 3.6b1 since feature improvements are taking priority on my time to make the feature freeze deadline).
msg274044 -(view)Author: Xiang Zhang (xiang.zhang)*(Python committer)Date: 2016-08-31 17:27
It's okay. Glad that you spare some time to reply. :)
msg274923 -(view)Author: Roundup Robot (python-dev)(Python triager)Date: 2016-09-08 00:01
New changeset96c6816825dc by Brett Cannon in branch 'default':Issue#27911: Remove some unnecessary error checks in import.c.https://hg.python.org/cpython/rev/96c6816825dc
msg274924 -(view)Author: Brett Cannon (brett.cannon)*(Python committer)Date: 2016-09-08 00:01
Thanks for the patch!
History
DateUserActionArgs
2022-04-11 14:58:35adminsetgithub: 72098
2016-09-08 00:01:37brett.cannonsetstatus: open -> closed
resolution: fixed
messages: +msg274924

stage: resolved
2016-09-08 00:01:12python-devsetnosy: +python-dev
messages: +msg274923
2016-08-31 17:27:57xiang.zhangsetmessages: +msg274044
2016-08-31 17:25:16brett.cannonsetassignee:brett.cannon
messages: +msg274043
2016-08-31 11:22:37xiang.zhangcreate
Supported byThe Python Software Foundation,
Powered byRoundup
Copyright © 1990-2022,Python Software Foundation
Legal Statements

[8]ページ先頭

©2009-2026 Movatter.jp