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 — Generalize `path_t` for C level optimizations · Issue #118263 · python/cpython

Generalize path_t for C level optimizations #118263

New issue
Copy link
New issue
Copy link
Closed
Closed
Generalize path_t for C level optimizations#118263
Copy link
Labels
type-featureA feature request or enhancementA feature request or enhancement

Description

@nineteendo
nineteendo
opened on Apr 25, 2024
Issue body actions

Feature or enhancement

Proposal:

Quoting @eryksun:

The implementation of path_t could be generalized to support fields to configure the converter to use wide regardless of platform, to allow null characters, to allow arbitrary length paths (e.g. no 32767 length limit on Windows), and a new field such as bytes_input to determine whether a path result has to be converted back to bytes. The option to always use a wide-character path is a generalization of the current behavior on Windows. Argument Clinic would be extended to support the new options. The implementations of _path_splitroot_ex(), _path_normpath(), and _path_abspath() (if adopted) would benefit, and also the _path_is*() helpers on Windows.

Has this already been discussed elsewhere?

This is a minor feature, which does not need previous discussion elsewhere

Links to previous discussion of this feature:

  • gh-102511: Add C implementation of os.path.splitroot() #118089

Linked PRs

  • gh-118263: Generalize path_t for C level optimizations #118355
  • [3.13] gh-118263: Add additional arguments to path_t (Argument Clinic type) in posixmodule (GH-118355) #119513
  • [3.12] gh-118263: Add additional arguments to path_t (Argument Clinic type) in posixmodule (GH-118355) #119608

Activity

  1. nineteendo
    added
    type-featureA feature request or enhancementA feature request or enhancement
    on Apr 25, 2024
  2. nineteendo commented on Apr 25, 2024

    @nineteendo
    nineteendo
    on Apr 25, 2024
    ContributorAuthor
    More actions

    cc @zooba, @serhiy-storchaka Sorry for the ping, I couldn't wait on your response, so I implemented it on my own.

  3. zooba
    added a commit that references this issue on May 24, 2024

    gh-118263: Add additional arguments to path_t (Argument Clinic type) …

    96b392d
  4. miss-islington
    added a commit that references this issue on May 24, 2024

    pythongh-118263: Add additional arguments to path_t (Argument Clinic …

    cc885e3
  5. zooba
    added a commit that references this issue on May 24, 2024

    gh-118263: Add additional arguments to path_t (Argument Clinic type) …

    cc38ee1
  6. nineteendo
    added a commit that references this issue on May 27, 2024

    pythongh-118263: Add additional arguments to path_t (Argument Clinic …

    a7f3e06
  7. zooba
    added a commit that references this issue on May 28, 2024

    gh-118263: Add additional arguments to path_t (Argument Clinic type) …

    09a85ea
  8. nineteendo
    closed this as completedon May 28, 2024
  9. estyxx
    added a commit that references this issue on Jul 17, 2024

    pythongh-118263: Add additional arguments to path_t (Argument Clinic …

    e810655
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 enhancementA feature request or enhancement

    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.