Issue40208
Created on 2020-04-06 17:05 by BTaskaya, last changed 2020-04-13 23:58 by corona10. This issue is now closed.
| Pull Requests | |||
|---|---|---|---|
| URL | Status | Linked | Edit |
| PR 19396 | merged | BTaskaya, 2020-04-06 17:12 | |
| Messages (6) | |||
|---|---|---|---|
| msg365874 - (view) | Author: Batuhan Taskaya (BTaskaya) * | Date: 2020-04-06 17:05 | |
SymbolTable's has_exec method deprecated 14 years ago (2006) with 2def557aba1aaa42b638f9bf95624b7e6929191c, it can be safely removed since there is no user of it. |
|||
| msg366176 - (view) | Author: Terry J. Reedy (terry.reedy) * | Date: 2020-04-11 01:35 | |
https://docs.python.org/3/library/symtable.html#symtable.SymbolTable.has_exec merely says "Return True if the block uses exec." This never happens since https://github.com/python/cpython/blob/3.8/Lib/symtable.py#L87 is "return False". I presume 'exec' refers to 2.x statement, not the 3.x function. The docstring on line 86 does say "Deprecated method." Guido, you changed these lines in 2006, in 2def557aba1aaa42b638f9bf95624b7e6929191c I presume you left them for the benefit of ported 2.7 code. Should the function be removed for 3.9 or left until 3.10? Your commit message referred to 'dead EXEC related constants'. Is there anything else that should be removed? |
|||
| msg366177 - (view) | Author: Terry J. Reedy (terry.reedy) * | Date: 2020-04-11 01:37 | |
Answer: no. The 2006 patch already removed OPT_EXEC and OPT_BARE_EXEC. |
|||
| msg366185 - (view) | Author: Guido van Rossum (gvanrossum) * | Date: 2020-04-11 03:33 | |
Yeah, this seems safe to remove. If somebody's code breaks, that will just help them remove some dead code. |
|||
| msg366350 - (view) | Author: Dong-hee Na (corona10) * | Date: 2020-04-13 23:51 | |
New changeset 990ea4200f05fcd8bce3de363f4d7745ae142385 by Batuhan Taşkaya in branch 'master': bpo-40208: Remove deprecated has_exec method of SymbolTable (GH-19396) https://github.com/python/cpython/commit/990ea4200f05fcd8bce3de363f4d7745ae142385 |
|||
| msg366351 - (view) | Author: Dong-hee Na (corona10) * | Date: 2020-04-13 23:58 | |
The deletion of this symtable.SymbolTable.has_exec is now landed. I would like to thank Batuhan and all the reviewers who have participated in this issue. It's time for us to say goodbye to symtable.SymbolTable.has_exec. |
|||
| History | |||
|---|---|---|---|
| Date | User | Action | Args |
| 2020-04-13 23:58:43 | corona10 | set | status: open -> closed resolution: fixed messages: + msg366351 stage: patch review -> resolved |
| 2020-04-13 23:51:38 | corona10 | set | nosy:
+ corona10 messages: + msg366350 |
| 2020-04-11 03:33:09 | gvanrossum | set | messages: + msg366185 |
| 2020-04-11 01:37:17 | terry.reedy | set | messages: + msg366177 |
| 2020-04-11 01:35:53 | terry.reedy | set | nosy:
+ gvanrossum, terry.reedy messages:
+ msg366176 |
| 2020-04-06 17:12:23 | BTaskaya | set | keywords:
+ patch stage: patch review pull_requests: + pull_request18758 |
| 2020-04-06 17:05:22 | BTaskaya | create | |