GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — PEP 737 implementation errors · Issue #117642 · python/cpython

PEP 737 implementation errors #117642

New issue
Copy link
New issue
Copy link
Closed
Closed
PEP 737 implementation errors#117642
Copy link
Labels
3.13only security fixesonly security fixestopic-C-APItype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

Description

@serhiy-storchaka
serhiy-storchaka
opened on Apr 8, 2024
Issue body actions

Bug report

For some reasons gh-111696 implemented %T# and %N# instead of %#T and %#N. It does not only contradict PEP 737, but does not match the general principles of prinf-like format, and can cause problems in future.

Tests, which were added in gh-111696, were removed in gh-116417, so now this feature is not tested.

Linked PRs

  • gh-117642: Fix PEP 737 implementation #117643

Activity

  1. serhiy-storchaka
    added
    type-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error
    topic-C-API
    3.13only security fixesonly security fixes
    on Apr 8, 2024
  2. serhiy-storchaka
    added 2 commits that reference this issue on Apr 8, 2024

    pythongh-117642: Fix PEP 737 implementation

    5d7cc9c

    gh-117642: Fix PEP 737 implementation (GH-117643)

    24a2bd0

  3. vstinner commented on Apr 8, 2024

    @vstinner
    vstinner
    on Apr 8, 2024
    Member
    More actions

    I suppose that the issue can now be closed? Or did you spot other implementation issues?

  4. serhiy-storchaka commented on Apr 8, 2024

    @serhiy-storchaka
    serhiy-storchaka
    on Apr 8, 2024
    MemberAuthor
    More actions

    Nothing more. I noticed a weird handling of # and when tried to update the tests, did not found them at place.

  5. serhiy-storchaka
    closed this as completedon Apr 8, 2024

  6. vstinner commented on Apr 8, 2024

    @vstinner
    vstinner
    on Apr 8, 2024
    Member
    More actions

    Thanks for the fix.

  7. diegorusso
    added a commit that references this issue on Apr 17, 2024

    pythongh-117642: Fix PEP 737 implementation (pythonGH-117643)

    7bfeb8b
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

    3.13only security fixesonly security fixestopic-C-APItype-bugAn unexpected behavior, bug, or errorAn unexpected behavior, bug, or error

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions
      You are visitor 01402503 — GeezerVison does not store the pages you look up.
      Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.