[Perl/perl5] 63fccd: regprop() is defined outside DEBUGGING

[email protected] (Karl Williamson via perl5-changes)
Newsgroups perl.perl5.changes
Message-ID <Perl/perl5/push/refs/heads/blead/[email protected]>
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 63fccd6eedc933b331c773378d9251453159d548
      https://github.com/Perl/perl5/commit/63fccd6eedc933b331c773378d9251453159d548
  Author: Karl Williamson <[email protected]>
  Date:   2026-03-14 (Sat, 14 Mar 2026)

  Changed paths:
    M embed.fnc
    M embed.h
    M proto.h

  Log Message:
  -----------
  regprop() is defined outside DEBUGGING

embed.fnc thought this function only existed in DEBUGGING, but in
reality it does; most of the contents are stubbed out unless DEBUGGING,
but it is callable without DEBUGGING.


  Commit: c63529a1cbb41c96a4456037c9408394f894d01b
      https://github.com/Perl/perl5/commit/c63529a1cbb41c96a4456037c9408394f894d01b
  Author: Karl Williamson <[email protected]>
  Date:   2026-03-14 (Sat, 14 Mar 2026)

  Changed paths:
    M embed.fnc
    M proto.h

  Log Message:
  -----------
  embed.fnc: Use actual formal parameter names

I discovered that these 19 functions had mismatched parameter names in
embed.fnc compared with the actual names in the function definition.
Despite not being good practice, this hasn't mattered so far, but will
in a future commit.


  Commit: 1c44630cd4854a002e5f7a2108ae87b0eeed8923
      https://github.com/Perl/perl5/commit/1c44630cd4854a002e5f7a2108ae87b0eeed8923
  Author: Karl Williamson <[email protected]>
  Date:   2026-03-14 (Sat, 14 Mar 2026)

  Changed paths:
    M av.c
    M builtin.c
    M deb.c
    M doio.c
    M doop.c
    M dquote.c
    M dump.c
    M gv.c
    M hv.c
    M inline.h
    M invlist_inline.h
    M locale.c
    M malloc.c
    M mg.c
    M mro_core.c
    M numeric.c
    M op.c
    M pad.c
    M peep.c
    M perl.c
    M perlio.c
    M pp.c
    M pp_ctl.c
    M pp_hot.c
    M pp_pack.c
    M pp_sort.c
    M pp_sys.c
    M reentr.c
    M regcomp.c
    M regcomp_debug.c
    M regcomp_invlist.c
    M regcomp_study.c
    M regcomp_trie.c
    M regen/reentr.pl
    M regexec.c
    M scope.c
    M sv.c
    M sv_inline.h
    M taint.c
    M toke.c
    M universal.c
    M utf8.c
    M util.c

  Log Message:
  -----------
  Move ARGS_ASSERT macros to top of their functions

Besides being more tidy, this makes sure that the asserts inside them
are executed before any attempt to use the parameters.  This will give
better error messages when those assertions are violated, instead of the
generic SEGFAULT that could occur previously.


Compare: https://github.com/Perl/perl5/compare/a772498751f0...1c44630cd485

To unsubscribe from these emails, change your notification settings at https://github.com/Perl/perl5/settings/notifications
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.