In perl.git, the branch smoke-me/khw-simple has been created
<https://perl5.git.perl.org/perl.git/commitdiff/c61a005ed5d77e1cd8c0a153eca7f35e8276a457?hp=0000000000000000000000000000000000000000>
at c61a005ed5d77e1cd8c0a153eca7f35e8276a457 (commit)
- Log -----------------------------------------------------------------
commit c61a005ed5d77e1cd8c0a153eca7f35e8276a457
Author: Karl Williamson <[email protected]>
Date: Wed May 29 19:11:04 2019 -0600
perlguts: Remove documentation about embed.fnc flags
embed.fnc now has sufficient comments within it. Anyone changing that
file will see those comments first. Having them redundantly explained
here adds little value, and is out-of-date. So just remove them, with a
ptr to embed.fnc
commit 1dd06e1e0e841a815b1ef7c52ee206f596c47bb0
Author: Karl Williamson <[email protected]>
Date: Wed May 29 19:10:19 2019 -0600
embed.fnc: Add more extensive comments
This revises the comments significantly to better inform the reader.
commit c1de5f2f96de12c4def32cfc6624271d7b7605ba
Author: Karl Williamson <[email protected]>
Date: Wed May 29 18:33:25 2019 -0600
Remove redundant info on =for apidoc lines
This information is already in embed.fnc, and we know it compiles. Some
of this information is now out-of-date. Get rid of it.
There was one bit of information that was (apparently) wrong in
embed.fnc. The apidoc line asked that there be no usage example
generated for newXS. I added that flag to the embed.fnc entry.
commit 10edb6153eea78662d8cefffdf6882aa4da569b4
Author: Karl Williamson <[email protected]>
Date: Wed May 29 16:06:51 2019 -0600
autodoc.pl: Use embed.fnc entry when available
Having two different pod definitions for the same function leads to
things being out of sync. This commit ignores the one in the '=for
apidoc' line if there is one in embed.fnc.
Doing so led to a bunch of changes in perlapi, showing that the apidoc
lines were inaccurate. The ones in embed.fnc must be accurate enough to
get perl to compile.
I added a warning for when there's a redundant entry, and in the next
commit I will remove the many such.
commit b2ef5e5b58898455bd57d96258ebcc4791ad40df
Author: Karl Williamson <[email protected]>
Date: Wed May 29 16:01:34 2019 -0600
autodoc.pl: Add N flag
This adds a check that the macro or function name for the pod entry is a
legitimate name, as almost all are. If this had been in effect, the bug
fixed by the previous commit would not have been necessary.
But there are a very few things that are documented that aren't strict
names. This adds a flag so that autodoc can be notified to not warn on
them. This will allow Devel::PPPort to not have to special case
situations like this, which it now does.
commit 3ff604309ff954c2755697a2cf782ebf556e9d79
Author: Karl Williamson <[email protected]>
Date: Wed May 29 15:58:20 2019 -0600
autodoc.pl: Check for misuse of the 'n' flag
This flag indicates that the macro has no arguments. It makes no sense
to use in some situations, which are now checked for.
This commit also documents the long existing 'n' flag.
commit 8d4a1bf37da86d9b6585f4e1a22723fc3d7be09f
Author: Karl Williamson <[email protected]>
Date: Wed May 29 15:50:45 2019 -0600
autodoc.pl: Make 's' flag independent of 'n'
The 's' flag is used by autodoc.pl to add a semi-colon to the end of a
usage example. This commit allows that to happen even if the macro has
parameters.
This allows two macros in scope.c to be properly documented. Previously
they weren't, and caused warnings in Devel::PPPort as a result.
This commit adds documentation to embed.fnc for the 's' flag, which has
long existed, undocumented.
commit f0a8728fb7c6e37e6c7bcb52f22ef650421af73c
Author: Karl Williamson <[email protected]>
Date: Wed May 29 15:41:23 2019 -0600
autodoc.pl: Notice the no-thread context flag
I realized reading the code that it would be possible for a function
definition to be wrongly output if it isn't called with a thread context
variable. The information for that is already in embed.fnc, and so we
just have to use it to avoid printing the variable if not present
commit 70a99663b332cbc0073838a0a0b56312baddb97e
Author: Karl Williamson <[email protected]>
Date: Wed May 29 15:28:44 2019 -0600
embed.fnc: Add flag for bypass macro existence
Sometimes a function is reduced to be a wrapper, and we want for code to
directly call the underlying one, but we retain the old 'Perl_foo'
function to avoid breaking code that used that form. I've tried various
kludges around that, but this seems more promising.
commit d9da0cef9fb5e14db836e5cb8ede3a8d8ad6728c
Author: Karl Williamson <[email protected]>
Date: Wed May 29 15:05:05 2019 -0600
autodoc.pl: Use O flag from embed.fnc for warning
This is for ancient 'perl_' forms of function calls, which is
deprecated. It turns out that there were a couple of entries that were
getting this warning when that form doesn't even exist.
commit c7853d153c3041ad15c0603e3f68be2ffac62fc7
Author: Karl Williamson <[email protected]>
Date: Wed May 29 14:49:57 2019 -0600
Let embed.fnc define the my_str_foo() fcns
We don't always define a Perl_ form for these, expecting all calls to be
made through the macro, which will use the native form on the systems
that have one.
Removing these extra #defines helps future commits
commit 6dcfc23dbe9a4912f37f5129e48c2140a66dff65
Author: Karl Williamson <[email protected]>
Date: Wed May 29 10:14:55 2019 -0600
embed.fnc: Change M flag to x
The M flag in embed.fnc and the x flag in the '=for apidoc' lines both
mean the same, thing, the entity may change, it is experimental.
Devel::PPPort combines both these things, and expects things to be
consistent.
This commit changes the 'M' in embed.fnc to be 'x'. Changing embed.fnc
and makedef.pl keeps the changes more contained, and lessens the chance
that some module will be adversely affected (I didn't see anything
likely in grepping cpan).
commit 54c2dd7564c2a419160a385547a4153f384fa60d
Author: Karl Williamson <[email protected]>
Date: Tue May 28 22:02:57 2019 -0600
embed.fnc: Change x flag to e
autodoc.pl has the x flag in the '=for apidoc' lines mean something
completely different, but Devel::PPPort combines both these lines and
the contents of embed.fnc into one file, expecting the flags to be
consistent.
This commit changes the 'e' in embed.fnc to be 'e', which means to not
export this function. Changing embed.fnc and makedef.pl keeps the
changes more contained, and lessens the chance that some module will be
adversely affected (I didn't see anything likely in grepping cpan).
commit 884a2f31d2c6c22929321f8f5cadd4430d2de2e5
Author: Karl Williamson <[email protected]>
Date: Tue May 28 20:28:07 2019 -0600
embed.fnc: Change s flag to S
autodoc.pl has the s flag in the '=for apidoc' lines mean something
completely different, but Devel::PPPort combines both these lines and
the contents of embed.fnc into one file, expecting the flags to be
consistent.
This commit changes the 's' in embed.fnc to be 'S', which means the
function is static and has an 'S_' prefix. Changing embed.fnc keeps the
changes more contained, and lessens the chance that some module will be
adversely affected (I didn't see anything likely in grepping cpan).
commit 605611188fd762eed56bae58e4ffec72a1dce2c3
Author: Karl Williamson <[email protected]>
Date: Tue May 28 20:52:07 2019 -0600
croak_memory_wrap is an inline function.
Update embed.fnc to correspond
commit 23ce803c96040340ab783f85d76a8a55103167d1
Author: Karl Williamson <[email protected]>
Date: Tue May 28 20:03:58 2019 -0600
embed.fnc: Change n flag to T
autodoc.pl has the n flag in the '=for apidoc' lines mean something
completely different, but Devel::PPPort combines both these lines and
the contents of embed.fnc into one file, expecting the flags to be
consistent.
This commit changes the 'n' in embed.fnc to be 'T', which means no
thread context passed. Changing embed.fnc keeps the changes more
contained, and lessens the chance that some module will be adversely
affected (I didn't see anything likely in grepping cpan).
commit 36d040e7cfae61b5c0dd1fdd89bd4a354591f194
Author: Karl Williamson <[email protected]>
Date: Tue May 28 12:55:19 2019 -0600
Add 'n' flag to various =for apidoc lines
This indicates to not output the macro with parentheses for parameters.
Currently that doesn't happen anyway, but a future commit will change
things so this is required (so that a bug can be fixed)
commit 959f9f6838baa4a45e9482ed30394833e89c2409
Author: Karl Williamson <[email protected]>
Date: Tue May 28 11:02:42 2019 -0600
autodoc.pl: Strip, lead/trail space from =for apidoc
The components of a =for apidoc entry can have spaces between the '|'
separators. Ignore them.
commit f844b914c8c80c9f2c61146d614e8d0f7429074b
Author: Karl Williamson <[email protected]>
Date: Tue May 28 10:42:21 2019 -0600
autodoc.pl: White-space only
This indents to correspond with the new block formed by the previous
commit.
commit e011cabc8fda46bbd2fc420178201767c1ce5fb5
Author: Karl Williamson <[email protected]>
Date: Tue May 28 10:35:57 2019 -0600
autodoc.pl: Refactor slightly
This breaks a chain of 'elsif' lines so that they are all now part of an
'else' from the first 'if'. This is in preparation for a later commit.
commit 60eb55e0749fe8a976cb5df434a1a6b99b195a6c
Author: Karl Williamson <[email protected]>
Date: Tue May 28 10:21:56 2019 -0600
autodoc.pl: Change name of variable
This makes it clear that the source for the lines being processed is
embed.fnc
commit d65cf4877169e2c5d132aaec84c9e71d30b497df
Author: Karl Williamson <[email protected]>
Date: Mon May 27 12:53:48 2019 -0600
JustPod_corpus.t: Suppress expected warns under harness
commit 713b82c0542974cdfe0e0750312a7de041f5ff63
Author: Karl Williamson <[email protected]>
Date: Wed May 22 10:23:50 2019 -0600
Bump VERSION to 3.37
-----------------------------------------------------------------------
--
Perl5 Master Repository
