Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Move runtime static init to its own header file. #90528

Closed
ericsnowcurrently opened this issue Jan 13, 2022 · 2 comments
Closed

Move runtime static init to its own header file. #90528

ericsnowcurrently opened this issue Jan 13, 2022 · 2 comments
Assignees
Labels
3.11 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs)

Comments

@ericsnowcurrently
Copy link
Member

BPO 46370
Nosy @ericsnowcurrently
PRs
  • bpo-46370: Move the static initializer for _PyRuntime to its own header file. #30587
  • Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.

    Show more details

    GitHub fields:

    assignee = 'https://github.com/ericsnowcurrently'
    closed_at = <Date 2022-01-13.22:55:08.909>
    created_at = <Date 2022-01-13.22:19:28.295>
    labels = ['interpreter-core', '3.11']
    title = 'Move runtime static init to its own header file.'
    updated_at = <Date 2022-01-13.22:55:08.909>
    user = 'https://github.com/ericsnowcurrently'

    bugs.python.org fields:

    activity = <Date 2022-01-13.22:55:08.909>
    actor = 'eric.snow'
    assignee = 'eric.snow'
    closed = True
    closed_date = <Date 2022-01-13.22:55:08.909>
    closer = 'eric.snow'
    components = ['Interpreter Core']
    creation = <Date 2022-01-13.22:19:28.295>
    creator = 'eric.snow'
    dependencies = []
    files = []
    hgrepos = []
    issue_num = 46370
    keywords = ['patch']
    message_count = 2.0
    messages = ['410529', '410530']
    nosy_count = 1.0
    nosy_names = ['eric.snow']
    pr_nums = ['30587']
    priority = 'normal'
    resolution = 'fixed'
    stage = 'resolved'
    status = 'closed'
    superseder = None
    type = None
    url = 'https://bugs.python.org/issue46370'
    versions = ['Python 3.11']

    @ericsnowcurrently
    Copy link
    Member Author

    The static initializer for _PyRuntime is currently defined in Include/internal/pycore_runtime.h. However, it is only needed by Python/pylifecycle.c (and Python/pystate.c for an optimization) and should only be used there. (Also, the initializer is quite large.) So I'm planning on moving it to it's own internal header file.

    @ericsnowcurrently ericsnowcurrently added the 3.11 only security fixes label Jan 13, 2022
    @ericsnowcurrently ericsnowcurrently self-assigned this Jan 13, 2022
    @ericsnowcurrently ericsnowcurrently added interpreter-core (Objects, Python, Grammar, and Parser dirs) 3.11 only security fixes labels Jan 13, 2022
    @ericsnowcurrently ericsnowcurrently self-assigned this Jan 13, 2022
    @ericsnowcurrently ericsnowcurrently added the interpreter-core (Objects, Python, Grammar, and Parser dirs) label Jan 13, 2022
    @ericsnowcurrently
    Copy link
    Member Author

    New changeset bc02eac by Eric Snow in branch 'main':
    bpo-46370: Move the static initializer for _PyRuntime to its own header file. (gh-30587)
    bc02eac

    @ezio-melotti ezio-melotti transferred this issue from another repository Apr 10, 2022
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
    Labels
    3.11 only security fixes interpreter-core (Objects, Python, Grammar, and Parser dirs)
    Projects
    None yet
    Development

    No branches or pull requests

    1 participant