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

Replacing %-formatting and string concatenation with f-strings in platform module · Issue #94713 · python/cpython · GitHub

Repository navigation

Replacing %-formatting and string concatenation with f-strings in platform module #94713

Closed as not planned

Description

A simple rewriting of some of the code to be more modern. Replacing %-formatting and string concatenation with f-strings and replacing while 1 with while True.

Activity

  1. changed the title [-]Replacing %-formatting and string concatenation with f-strings[/-] [+]Replacing %-formatting and string concatenation with f-strings in platform module[/+] on Jul 9, 2022
  2. stevendaprano commented on Jul 10, 2022

    Member

    We don't make a habit of re-writing working code just for the sake of re-writing it. %-formatting works, and is usually faster than "more modern" formatting, so that would be code churn that makes the code worse rather than better.

    Since this is just a cosmetic change that will probably hurt performance, I'm going to close this. If you benchmark the changes and find that they are significantly faster, feel free to reopen.

  3. added a commit that references this issue on Jul 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    type-featureA feature request or enhancement

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions


      Back | FazBrowse Home | New Git URL