develooper Front page | perl.perl5.changes | Postings from September 2019

[perl.git] branch smoke-me/khw-ppport created.v5.31.3-179-g7e57bb6e7c

From:
Karl Williamson
Date:
September 14, 2019 21:00
Subject:
[perl.git] branch smoke-me/khw-ppport created.v5.31.3-179-g7e57bb6e7c
Message ID:
E1i9F9X-0008IX-3M@git.dc.perl.space
In perl.git, the branch smoke-me/khw-ppport has been created

<https://perl5.git.perl.org/perl.git/commitdiff/7e57bb6e7c0b992fce3575b633bab091f5d0bd27?hp=0000000000000000000000000000000000000000>

        at  7e57bb6e7c0b992fce3575b633bab091f5d0bd27 (commit)

- Log -----------------------------------------------------------------
commit 7e57bb6e7c0b992fce3575b633bab091f5d0bd27
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:34:23 2019 -0600

    regen/embed.pl: Enforce some flag restrictions
    
    embed.fnc now has been updated to not have these flaws, so can now
    enforce them for the future.

commit adb4944d4dbb41b8e8c149d09499e6127cf1abe2
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:33:16 2019 -0600

    regen/embed.pl: Update a branch test
    
    The two flags are now mutually exclusive, so no need to test for both.

commit 458d6b63de34378937d403c3ae60030c515d9879
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:30:18 2019 -0600

    embed.fnc: M and o flags don't make sense together
    
    The M flag does what the o flag except add an incorrect warning.  So
    remove the 'o' which gets rid of the warning.

commit f4b44dc629493a72969f45846f6f93694c5bd91d
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:26:05 2019 -0600

    embed.fnc: 'm' and 'p' flags don't make sense together
    
    This is a macro, so doesn't have a Perl_ long form function call.
    Also forbid 'm' and 'S', but there weren't any current uses of that.

commit 9567337a0329d51a98d0e56e479108228c9c50ad
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:16:19 2019 -0600

    perlapi: Properly document Perl_custom_op_xop()
    
    It requires the prefix and a thread context parameter.

commit 322d6ba0cf3ce68a37dc031704c183ef18aea99e
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:11:47 2019 -0600

    autodoc.pl: Special case macros whose name begins with Perl_
    
    The usage example for these will have the thread context parameter
    unless there is no thread context.

commit ca57ec71bc7cb4fc86a190052e1b5503f831121a
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Sep 14 14:06:51 2019 -0600

    embed.fnc: Add some comment about name space pollution

commit 9f6478fd16201950d4eae6c37220e109ba6e0bb0
Author: Karl Williamson <khw@cpan.org>
Date:   Thu Sep 12 14:31:47 2019 -0600

    embed.fnc: E, X flags don't make sense with m
    
    Macros can't currently be restricted in scope.

commit d5ba400fb211a3edea1eea922b9ab95aed188823
Author: Karl Williamson <khw@cpan.org>
Date:   Thu Sep 12 14:24:48 2019 -0600

    Document IS_SAFE_SYSCALL

commit b11eb9587566699bdfc0c80287ce1159ab842cf0
Author: Karl Williamson <khw@cpan.org>
Date:   Thu Sep 12 14:23:18 2019 -0600

    perlapi: Improve description of is_safe_syscall()
    
    Two parameters weren't described, nor the warnings category used.

commit 83dc7bde4432947524f8d88f164082eda7bb1475
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 22:08:24 2019 -0600

    embed.fnc: Add p flag for is_utf8_non_invariant_string()
    
    This is because it has the X flag, which means the function is visible
    on ELF systems.

commit 63e1180d937ab946057039eea7c9a80858734c27
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 20:33:05 2019 -0600

    Add embed.fnc entry for ReANY
    
    This inline function should have an entry.

commit 81150d236fcd8af6c814d0b3df3af3a95d7820fd
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 18:18:33 2019 -0600

    Add embed.fnc entries for TOPMARK, POPMARK

commit 38dc1c060fbd22b24616f3a2475545ed3af7c2f8
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 17:37:23 2019 -0600

    Add Perl_ to  SvAMAGIC _off,_on names
    
    These function names need a Perl_ prefix to avoid namespace pollution.

commit 71d6a3e9e2d3e947ff2343a9dcc26f3f94fa51a6
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 17:24:38 2019 -0600

    Add embed.fnc entries for Cv inline fcns
    
    and change the name of one.  CvDEPTH shouldn't have a trailing 'p' to
    indicate private memeber access.  It may do so internally, but the name
    shouldn't indicate that.

commit 2932f4de79ddf62184a9d87c977d76cec6fdbe9e
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 17:17:18 2019 -0600

    mathoms.c: Add comment

commit feba79baa9a4aed5ee476ffebf2a0f5adf4a65fd
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 17:08:25 2019 -0600

    Fix up get_regex_charset_name()
    
    This inline function should not be visible outside certain core (and
    ext/re) functions.  Make that happen.

commit a745de7d5747f057f8fb3436f8b637131551ce77
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 17:03:29 2019 -0600

    inline.h: Move some fcn '{' to column 1
    
    Traditionally, functions in perl have been written so their initial '{'
    is in column 1 after the prototype.  This makes grepping easier.  This
    commit brings 3 functions in this file into conformance.

commit 37db4c6cc93e0451ddf25a5792ffc7939c4c1ad0
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 15:56:11 2019 -0600

    Make PadnameIN_SCOPE() name begin with S_
    
    All functions should have a prefix.

commit f167cf5717661f356b7628fc1179d89704fd14d2
Author: Karl Williamson <khw@cpan.org>
Date:   Tue Sep 10 12:37:28 2019 -0600

    Some SvREFCNT_foo are no longer macros
    
    but are inline functions.  They should be listed in embed.fnc.

commit dae884318eea98ab19a7e9572e6812560f98a969
Author: Karl Williamson <khw@cpan.org>
Date:   Tue Sep 10 11:51:40 2019 -0600

    Strip leading underscore _variant_byte_number()
    
    Names with leading underscores are reserved for the C implementers.

commit 3675945c0b8c7497e5cd2b29f6437476c54e1100
Author: Karl Williamson <khw@cpan.org>
Date:   Tue Sep 10 11:12:59 2019 -0600

     Change name of _utf8_to_uvchr_buf()
    
    A function name with a leading underscore is not legal in C.  Instead
    add a suffix to differentiate this name from an otherwise identical one.

commit 896eba13cbb45bedd7d310091c6e8ae5ec02d4eb
Author: Karl Williamson <khw@cpan.org>
Date:   Tue Sep 10 10:59:04 2019 -0600

    Strip leading underscore from is_utf8_char_helper()
    
    Leading underscored name are reserved for the C implementers

commit 828b81d38ea8c5de72c6aba5a0617e4ef759c020
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Sep 11 15:20:45 2019 -0600

    inline.h: Change fcn name prefix from S_ to Perl_
    
    This is being done only for those functions that don't have a guard
    preventing them from being seen outside of the Perl core.
    
    Talking to Tony Cook, we agreed that this was a good idea for two
    reasons:
    
    1) The 'Perl_' prefix does not pollute XS caller's name space.  The 'S_'
       one could be argued that it doesn't much either, but it does more so
       than 'Perl_', and the next reason is the clincher:
    2) It allows us to change our minds about whether a function should be
       static inline or not, without affecting callers who use the Perl_
       form, which they would be accustomed to anyway if they're using the
       full name form.

commit 28d0fa765a265a5ee0d9ebabd76be97cec35f6b2
Author: Karl Williamson <khw@cpan.org>
Date:   Mon Sep 9 11:53:55 2019 -0600

    regen/embed.pl: Generally generate an ARGS_ASSERT
    
    Prior to this commit, if you changed a pointer argument to allow it
    being a NULL, you had to go in and also remove the function's
    PERL_ARGS_ASSERT macro call.  Now, such macros are generally defined
    even if they are empty, so you can add such a call to any function, and
    it will always compile.  If there is no assertion needed, the macro
    expands to nothing.
    
    Thus now, the use of these macros is optional when no assert is needed;
    but still required by the porting test args_assert.t when an assert is
    needed.
    
    The generation of such a macro (when its use is optional) can be
    suppressed for individual entries by the new G flag in embed.fnc, also
    added by this commit.

commit a4eef7cafa87d827591a7f7244baca0a9636d67c
Author: Karl Williamson <khw@cpan.org>
Date:   Mon Sep 9 11:43:18 2019 -0600

    regen/embed.pl: Wrap #if..#endif differently
    
    By surrounding the interior stuff with these in one statement, it makes
    it easier to change that interior stuff without having dangling
    statements.
    
    This makes no difference in the current generated output

commit 261dfeff263efc3cccc6425037e602c389ec204e
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Aug 31 20:58:57 2019 -0600

    embed.fnc: Mark atof2, atof3 for internal use only
    
    Now that we have Strtod, my_strtod, those should be used instead.

commit d2bc015a36bef6c10b109c280858a59e5186ce1a
Author: Karl Williamson <khw@cpan.org>
Date:   Sun Jul 21 14:02:31 2019 -0600

    sv.h: SVt_INVLIST is core-only

commit 591783f42091f1ed7d039a911ee7f24445f00df0
Author: Karl Williamson <khw@cpan.org>
Date:   Fri Aug 9 11:41:07 2019 -0600

    embed.fnc: Change some 'x' marked fcns to 'C'
    
    The previous commit added the C flag, which is better to use in many
    instances than x.  This commit changes the x entries in embed.fnc that
    I'm pretty sure would be better off being C, and removes some 'x'
    entries for internal only functions

commit c45b1ba697b595bb31d5a7a695a7d1ad89216811
Author: Karl Williamson <khw@cpan.org>
Date:   Sun Jul 21 13:20:55 2019 -0600

    Add 'C' flag to embed.fnc, for "core-only" fcns
    
    Previously the x flag was (mis-)used for this purpose.  Some functions
    that are accessible by XS writers shouldn't be used by them, such as
    helper functions that an inline public function calls.  This new flag
    alows that to be specified, without having to mark the function as
    experimental, which could be misleading to later core maintainers

commit 92e466f62924e7ddb555788950c3f797e1101250
Author: Karl Williamson <khw@cpan.org>
Date:   Sat Aug 31 14:30:29 2019 -0600

    embed.fnc: Clarify comments, and note 'u' flag exceptions
    
    The next release of Devel::PPPort will know how to handle all these
    unorthodox functions, so no need to declare them as such, and in fact
    they shouldn't be declared so.

commit a3491ab7b6246ab4fced79631b6999ae0950e2ce
Author: Karl Williamson <khw@cpan.org>
Date:   Wed Jun 26 13:02:35 2019 -0600

    Configure

commit 15535e619b2d9b2e5ff217f715da838c2be989c0
Author: Karl Williamson <khw@cpan.org>
Date:   Mon Nov 26 20:16:09 2018 -0700

    XXX need to do process; figure name Configure Fix alignment needed probe

-----------------------------------------------------------------------

-- 
Perl5 Master Repository



nntp.perl.org: Perl Programming lists via nntp and http.
Comments to Ask Bjørn Hansen at ask@perl.org | Group listing | About