Branch: refs/heads/blead
Home: https://github.com/Perl/perl5
Commit: b8837dad4db9806bc61659d5230a35b90655c07b
https://github.com/Perl/perl5/commit/b8837dad4db9806bc61659d5230a35b90655c07b
Author: Yves Orton <[email protected]>
Date: 2023-02-19 (Sun, 19 Feb 2023)
Changed paths:
M embed.fnc
M regen/tidy_embed.pl
Log Message:
-----------
embed.fnc - sort entries alphabetically by function name.
This is actually a library sort (lc with underbars removed), followed by a
lexicographical sort. Comment lines are sticky to the line that follows
them.
Somehow the original version of this patch was missed in my earlier work on
tidy_embed.pl, I think I messed up a rebase somehow. I noticed it was missing
when I realized that new entries werent being sorted into place correctly.
While this patch creates a fair bit of churn in the file right now, long
term it will make it easier to use. Also note that the *output* files
have not changed, which validates that the patch did not break anything.
Commit: d6a7165bd44b5bdf2dc544680b8f167b6ad3351f
https://github.com/Perl/perl5/commit/d6a7165bd44b5bdf2dc544680b8f167b6ad3351f
Author: Yves Orton <[email protected]>
Date: 2023-02-19 (Sun, 19 Feb 2023)
Changed paths:
M embed.fnc
M regen/HeaderParser.pm
M regen/embed.pl
Log Message:
-----------
embed.pl - sort and dedupe flags in embef.fnc as part of tidy
This ensures we use a canonical string for each possible flag variant,
which makes it easier to search for flags with a given flag signature.
It also exposed a mutex bug in flag handling which caused
PerlEnv_putenv to be improperly marked as static, when it is in fact
static inline.
To validate there aren't any issues like this remaining in the script
I set it up so the flags were shuffled during processing and ran
embed.pl in a loop for a while and none of the output files changed,
so I assume there are no further such issues.
This patch also includes some basic validation of the flags so that if
someone misses a line continuation the following lines are not treated
as a new definition without any flags. I also ran perltidy on it
according to the rules contained within the file.
Compare: https://github.com/Perl/perl5/compare/481446ecbbb0...d6a7165bd44b
