Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 353572705ee571443ddead80d6e5b5df6ad49f53
      
https://github.com/Perl/perl5/commit/353572705ee571443ddead80d6e5b5df6ad49f53
  Author: Yves Orton <demer...@gmail.com>
  Date:   2024-02-17 (Sat, 17 Feb 2024)

  Changed paths:
    M Makefile.SH

  Log Message:
  -----------
  Makefile.SH - when building with -Uusedl we do not need to build re.xs

This fixes make test_reonly when building under -Uusedl. We do not need
to call make_ext.pl on the re extension, as it will be statically linked
anyway.


  Commit: 52d02fb471fbbe62deeab0837c64dcfc86ac54b2
      
https://github.com/Perl/perl5/commit/52d02fb471fbbe62deeab0837c64dcfc86ac54b2
  Author: Yves Orton <demer...@gmail.com>
  Date:   2024-02-17 (Sat, 17 Feb 2024)

  Changed paths:
    M regcomp_internal.h

  Log Message:
  -----------
  regcomp_internal.h - handle -Uusedl in RExC_state_t

Under -Uusedl there were two problems: the first was that we were
conditionally adding fields based on whether DEBUGGING was true, but
this does not work properly when do a static build as we use the same
data structure definition for DEBUGGING and non-DEBUGGING builds as we
compile both into the same library. So in this case we just always
include the additional fields.

We probably could even just ALWAYS include them, as we normally only
have one of these structs around so the unused storage for these fields
would not really be a burden.

The second problem was that the debugging fields were not at the end of
the structure. This meant that code compiled under DEBUGGING and not
would access the same memory for different purposes.

This patch makes it so we include these fields if we are building under
DEBUGGING, OR if we are doing a static build. It also moves these fields
to the end so that there is no ambiguity about which memory is used
for what.


Compare: https://github.com/Perl/perl5/compare/a8c98abf827e...52d02fb471fb

To unsubscribe from these emails, change your notification settings at 
https://github.com/Perl/perl5/settings/notifications

Reply via email to