Skip to content

Conversation

khwilliamson
Copy link
Contributor

@khwilliamson khwilliamson commented Aug 21, 2025

These are case statement values used in switch() statements in several core .c files. These are internal details about how we currently implement things. Exposing them to the outside world risks being stuck in this implementation.

  • This set of changes does not require a perldelta entry.

@jkeenan
Copy link
Contributor

jkeenan commented Aug 21, 2025

These should not be used outside core

And ... why not?

These are case statement values used in switch() statements in several
core .c files.  These are internal details about how we currently
implement things.  Exposing them to the outside world risks being stuck
in this implementation.
@khwilliamson khwilliamson force-pushed the restrict_keywords_to_core branch from 19dada7 to e80521b Compare August 24, 2025 15:00
@Leont
Copy link
Contributor

Leont commented Aug 24, 2025

A quick grep suggests at least one CPAN dist (warnings::unused) would be broken by this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants