FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

gh-102498 Clean up unused variables and imports in the email module by JosephSBoyle · Pull Request #102482 · python/cpython · GitHub

/ cpython Public

gh-102498 Clean up unused variables and imports in the email module - #102482

Merged
warsaw merged 16 commits into
python:mainfrom
JosephSBoyle:fix-unused-vars-in-email
Apr 24, 2023
ConversationCommits (16)ChecksFiles changed
Merged

gh-102498 Clean up unused variables and imports in the email module #102482
warsaw merged 16 commits into
python:mainfrom
JosephSBoyle:fix-unused-vars-in-email

Conversation

JosephSBoyle commented Mar 6, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor

Found some unused vars using the following pyflakes command $ python3.12 -m pyflakes Lib/email/. Inspired by @AlexWaygood 's use of the tool to find issues in the test_typing.py file (can't seem to find the exact issue).

Cleanup some unused variables and imports in the email module.

In text.py I replaced a pagebreak character (U+000C) with a newline. The latter doesn't seem to be rendered by GH. VScode shows it, as will most editors I imagine.

GH Issue: #102498

JosephSBoyle requested a review from a team as a code owner March 6, 2023 21:48

warsaw commented Mar 7, 2023

Copy link
Copy Markdown
Member

Thanks for the cleanup. I can review it in more detail after a few housekeeping updates have been made (updating the branch to current HEAD, creating an issue, and adding an issue number).

You can just delete the page feed line (see PEP 8, two blank lines only between top level constructs). The page feeds are a historical holdover for when this library was developed independently.

JosephSBoyle mentioned this pull request Mar 7, 2023
Closed
JosephSBoyle changed the title Clean up unused variables and imports in the email module Clean up unused variables and imports in the email module #102498 Mar 7, 2023
JosephSBoyle force-pushed the fix-unused-vars-in-email branch from 2c31a05 to dc7176e Compare March 7, 2023 15:31

hugovk commented Mar 7, 2023

Copy link
Copy Markdown
Member

To appease Bedevere about the issue number, please prefix the PR title as "gh-XXXXX:"

JosephSBoyle changed the title Clean up unused variables and imports in the email module #102498 gh-102498 Clean up unused variables and imports in the email module Mar 7, 2023

JosephSBoyle commented Mar 7, 2023

Copy link
Copy Markdown
Contributor Author

Hi @warsaw, thanks for the quick response. I've added an issue, rebased to the current HEAD and removed that extra line as you requested. Should be good for a review now.

@hugovk cheers for the headsup r.e the issue number.

terryjreedy commented Mar 7, 2023

Copy link
Copy Markdown
Member

If you used a program/linter to find these, please add 'found by xxxx' to the first sentence in the message.

hugovk commented Mar 7, 2023

Copy link
Copy Markdown
Member

Also an unused msg here (found by Flake8):

cpython/Lib/email/feedparser.py

Line 269 in 4a3ea1f

msg = self._pop_message()

JosephSBoyle commented Mar 7, 2023

Copy link
Copy Markdown
Contributor Author

There are 45 pagebreaks remaining in the entire cpython repo, 34 of which are in email. I'm happy to replace them with newlines / delete them as appropriate per pep-8 if people want them gone. Might be something for another issue though:)

AlexWaygood commented Mar 7, 2023 •
edited
Loading

Copy link
Copy Markdown
Member

There are 45 pagebreaks remaining in the entire cpython repo, 34 of which are in email. I'm happy to replace them with newlines / delete them as appropriate per pep-8 if people want them gone. Might be something for another issue though:)

I agree with @warsaw that it's now time for these to go, but it would be better to remove all of them from the stdlib at once, in a separate PR, I think, rather than in this PR.

(Not requesting a change to this PR; it's fine to remove the one you're already removing, in my opinion, since you're already touching that file :)

AlexWaygood commented Mar 7, 2023

Copy link
Copy Markdown
Member

Inspired by @AlexWaygood 's use of the tool to find issues in the test_typing.py file (can't seem to find the exact issue).

This is the one:

JosephSBoyle mentioned this pull request Mar 7, 2023
Closed
JosephSBoyle force-pushed the fix-unused-vars-in-email branch from aa6402a to 76f5222 Compare March 8, 2023 11:01

JosephSBoyle commented Mar 9, 2023 •
edited
Loading

Copy link
Copy Markdown
Contributor Author

I appear to have duplicated the commits during the merge somehow. I would rebase and drop them but I'm under the impression that force-pushing is discouraged. If we merge this PR they'll all be squashed anyways as I understand it.. Let me know if dropping the duplicated commits is preferable and I can do that:)

hugovk commented Mar 9, 2023

Copy link
Copy Markdown
Member

Correct, all PRs are squashed and merged, so don't worry about the individual commits too much. 👍

JosephSBoyle commented Apr 22, 2023

Copy link
Copy Markdown
Contributor Author

Hi @warsaw. Could you please take a look at this when you get a chance?

warsaw enabled auto-merge (squash) April 24, 2023 18:44
warsaw merged commit 04ea048 into python:main Apr 24, 2023
carljm added a commit to carljm/cpython that referenced this pull request Apr 24, 2023
Merge branch 'main' into superopt
19b8025 
* main: (53 commits)
  pythongh-102498 Clean up unused variables and imports in the email module  (python#102482)
  pythongh-99184: Bypass instance attribute access in `repr` of `weakref.ref` (python#99244)
  pythongh-99032: datetime docs: Encoding is no longer relevant (python#93365)
  pythongh-94300: Update datetime.strptime documentation (python#95318)
  pythongh-103776: Remove explicit uses of $(SHELL) from Makefile (pythonGH-103778)
  pythongh-87092: fix a few cases of incorrect error handling in compiler (python#103456)
  pythonGH-103727: Avoid advancing tokenizer too far in f-string mode (pythonGH-103775)
  Revert "Add tests for empty range equality (python#103751)" (python#103770)
  pythongh-94518: Port 23-argument `_posixsubprocess.fork_exec` to Argument Clinic (python#94519)
  pythonGH-65022: Fix description of copyreg.pickle function (python#102656)
  pythongh-103323: Get the "Current" Thread State from a Thread-Local Variable (pythongh-103324)
  pythongh-91687: modernize dataclass example typing (python#103773)
  pythongh-103746: Test `types.UnionType` and `Literal` types together (python#103747)
  pythongh-103765: Fix 'Warning: py:class reference target not found: ModuleSpec' (pythonGH-103769)
  pythongh-87452: Improve the Popen.returncode docs
  Removed unnecessary escaping of asterisks (python#103714)
  pythonGH-102973: Slim down Fedora packages in the dev container (python#103283)
  pythongh-103091: Add PyUnstable_Type_AssignVersionTag (python#103095)
  Add tests for empty range equality (python#103751)
  pythongh-103712: Increase the length of the type name in AttributeError messages (python#103713)
  ...
carljm added a commit to carljm/cpython that referenced this pull request Apr 24, 2023
Merge branch 'superopt' into superopt_spec
1972ca4 
* superopt: (82 commits)
  pythongh-101517: fix line number propagation in code generated for except* (python#103550)
  pythongh-103780: Use patch instead of mock in asyncio unix events test (python#103782)
  pythongh-102498 Clean up unused variables and imports in the email module  (python#102482)
  pythongh-99184: Bypass instance attribute access in `repr` of `weakref.ref` (python#99244)
  pythongh-99032: datetime docs: Encoding is no longer relevant (python#93365)
  pythongh-94300: Update datetime.strptime documentation (python#95318)
  pythongh-103776: Remove explicit uses of $(SHELL) from Makefile (pythonGH-103778)
  pythongh-87092: fix a few cases of incorrect error handling in compiler (python#103456)
  pythonGH-103727: Avoid advancing tokenizer too far in f-string mode (pythonGH-103775)
  Revert "Add tests for empty range equality (python#103751)" (python#103770)
  pythongh-94518: Port 23-argument `_posixsubprocess.fork_exec` to Argument Clinic (python#94519)
  pythonGH-65022: Fix description of copyreg.pickle function (python#102656)
  pythongh-103323: Get the "Current" Thread State from a Thread-Local Variable (pythongh-103324)
  pythongh-91687: modernize dataclass example typing (python#103773)
  pythongh-103746: Test `types.UnionType` and `Literal` types together (python#103747)
  pythongh-103765: Fix 'Warning: py:class reference target not found: ModuleSpec' (pythonGH-103769)
  pythongh-87452: Improve the Popen.returncode docs
  Removed unnecessary escaping of asterisks (python#103714)
  pythonGH-102973: Slim down Fedora packages in the dev container (python#103283)
  pythongh-103091: Add PyUnstable_Type_AssignVersionTag (python#103095)
  ...
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

warsaw warsaw approved these changes

Assignees

No one assigned

Labels

skip news topic-email

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

6 participants

Back | FazBrowse Home | New Git URL