This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

classification
Title: Remove the semicolon in source code
Type: enhancement Stage: resolved
Components: Library (Lib) Versions: Python 2.7
process
Status: closed Resolution: not a bug
Dependencies: Superseder:
Assigned To: Nosy List: ibigbug, martin.panter, rhettinger
Priority: normal Keywords: patch

Created on 2016-11-21 16:35 by ibigbug, last changed 2022-04-11 14:58 by admin. This issue is now closed.

Files
File name Uploaded Description Edit
mywork.patch ibigbug, 2016-11-21 16:35 the corresponding patch review
Repositories containing patches
2.7
Messages (5)
msg281378 - (view) Author: Yuwei Ba (ibigbug) * Date: 2016-11-21 16:35
As we do not often use semicolons in Python world. I hope this historical semi would be cleaned in a future release.

Source: Lib/httplib.py:1117
msg281425 - (view) Author: Martin Panter (martin.panter) * (Python committer) Date: 2016-11-22 01:12
This was added to 2.7 in r68532 (Issue 4879), but the semicolon never made it to the Python 3 branch that I can see, so I don’t think there is anything to fix there. Your patch is against the 2.7 branch, which is only open for bug fixes now, and I don’t think this qualifies, sorry :)
msg281433 - (view) Author: Yuwei Ba (ibigbug) * Date: 2016-11-22 05:24
I know this is not a functional 'fix', though it's the fact that there should not be a semicolon. It can make user confusing. There might be another user who found this semicolon and create an issue again since there are still lots of Python27 codes running on tons of machines.

So if possible, I'll persist on this patch to be applied. Though it's not so urgent, it's a right thing. ;)
msg281438 - (view) Author: Raymond Hettinger (rhettinger) * (Python committer) Date: 2016-11-22 07:01
Sorry Yuwei, but this is irrelevant and not appropriate for a Python 2.7 backport at this time.
msg281440 - (view) Author: Yuwei Ba (ibigbug) * Date: 2016-11-22 08:00
Okay.

Thanks you two fellows!
History
Date User Action Args
2022-04-11 14:58:39adminsetgithub: 72952
2016-11-22 08:10:17xiang.zhangsetstage: resolved
2016-11-22 08:00:05ibigbugsetmessages: + msg281440
2016-11-22 07:01:18rhettingersetstatus: open -> closed
nosy: + rhettinger
messages: + msg281438

2016-11-22 05:24:55ibigbugsetstatus: closed -> open
resolution: rejected -> not a bug
messages: + msg281433
2016-11-22 01:12:16martin.pantersetstatus: open -> closed

nosy: + martin.panter
messages: + msg281425

resolution: rejected
2016-11-21 16:35:39ibigbugcreate