embed.fnc: Update comments #24051
Open
embed.fnc: Update comments #24051
+80
−69
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Contributor
khwilliamson
commented
Jan 1, 2026
khwilliamson
commented
- This set of changes does not require a perldelta entry.
These haven't been true since autodoc.pl was changed in 5.38
This hasn't been true since the automatic formatting/sorting of embed.fnc was instituted in 5.38
This warned about a 5.31 change. That's long enough ago that people have gotten used to the new scheme.
by moving it to the top of the file and making it SHOUT
This file was originally written to handle functions and the short name macros that call them. But its contents have gradually been expanded over the years, without fully updating the comments to reflect that. This commit rewords things so that text that refers to more than just functions is generalized.
92dcf59 changed newly-created macros to be affected by the apidoc visibility flags. This commit updates the comments in embed.fnc to reflect that.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.