Branch: refs/heads/yves/RCPV_macros_and_docs
  Home:   https://github.com/Perl/perl5
  Commit: 430b2feb52b9e85ce5f3c36d05819f38bb9caa9b
      
https://github.com/Perl/perl5/commit/430b2feb52b9e85ce5f3c36d05819f38bb9caa9b
  Author: Yves Orton <[email protected]>
  Date:   2023-03-04 (Sat, 04 Mar 2023)

  Changed paths:
    M scope.c

  Log Message:
  -----------
  scope.c - fix typo


  Commit: d7cb837fe522887bcaedfa6c06ae69ec6722fdd8
      
https://github.com/Perl/perl5/commit/d7cb837fe522887bcaedfa6c06ae69ec6722fdd8
  Author: Yves Orton <[email protected]>
  Date:   2023-03-04 (Sat, 04 Mar 2023)

  Changed paths:
    M cop.h
    M embed.fnc
    M embed.h
    M pod/perlguts.pod
    M proto.h
    M regen/scope_types.pl
    M scope.c
    M scope.h
    M scope_types.h
    M sv.c

  Log Message:
  -----------
  scope.c - improved RCPV support, SAVERCPV SAVEFREERCPV

I misnamed some of the RCPV stuff when it was introduced. The macro
which I called SAVERCPVFREE should have been called SAVEFREERCPV,
and there should also have been a SAVERCPV macro.

There also should have been RCPV_REFCNT_dec() and RCPV_REFCNT_inc()
macros.

There was also missing documentation. This patch should fix all of these
issues. Since this functionality has never been released in a production
perl it should be fine to do these renames, nothing out there should use
these macros yet.

I noticed these oversights while fixing Scope::Upper, see also:
https://github.com/Perl/perl5/issues/20861
https://rt.cpan.org/Ticket/Display.html?id=146897


Compare: https://github.com/Perl/perl5/compare/430b2feb52b9%5E...d7cb837fe522

Reply via email to