Skip to content

Conversation

@khwilliamson
Copy link
Contributor

See #22291

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

@jkeenan
Copy link
Contributor

jkeenan commented Jul 17, 2025

@khwilliamson, this p.r. was labeled "documentation" on May 27 -- but I think that perhaps is misleading, given that no POD is changed. It's a couple of small changes to embed.fnc. Can you clarify? Also, who should be reviewing this p.r.?

@khwilliamson
Copy link
Contributor Author

It changes the generated perlapi.pod and perlintern.pod by moving the mention of these functions from the former to the latter.

I don't know who should review it. Since no one has in this interval, I'm tempted to merge it.

@Leont
Copy link
Contributor

Leont commented Jul 17, 2025

These functions should not be downgraded from API to core-only. They are needed specifically in threading implementations (such as threads.pm and Thread::CSP). And arguably for some obscure inter-thread communication modules. They should not be needed for any module whose core-business isn't threading.

@khwilliamson khwilliamson deleted the clone_not_api branch July 17, 2025 18:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants