develooper Front page | perl.perl5.changes | Postings from August 2022

[Perl/perl5] 58411b: hv_func.h - change _PERL style macros to PVT_styl...

From:
Yves Orton via perl5-changes
Date:
August 9, 2022 12:08
Subject:
[Perl/perl5] 58411b: hv_func.h - change _PERL style macros to PVT_styl...
Message ID:
Perl/perl5/push/refs/heads/blead/f7e7b4-00b420@github.com
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 58411bc7381d696a6aa53ea5106866fc975fe9ae
      https://github.com/Perl/perl5/commit/58411bc7381d696a6aa53ea5106866fc975fe9ae
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M hv_func.h
    M perlvars.h

  Log Message:
  -----------
  hv_func.h - change _PERL style macros to PVT_ style macros

C reserves identifiers starting with underbar, but I didnt realize that this
applies to defines as well when I set up this file. This renames the private
macros to start with PVT. It maintains the distinction between _PERL and __PERL
by converting them to PVT_PERL and PVT__PERL.


  Commit: 86d84dd06fe16998648c41a393a25a59b9c470b7
      https://github.com/Perl/perl5/commit/86d84dd06fe16998648c41a393a25a59b9c470b7
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M INSTALL
    M hv_func.h
    M perl.c

  Log Message:
  -----------
  perl.c - remove code for reporting state of non-existent defines in -V

Most of the hash based defines are no longer supported, so remove them.
Also at the same time introduce a new simpler way to track which hash
function we are using. Also add the info about if SBOX32 is in use.

This removes the need to keep the list of hash functions supported in
two places, hv_func.h and perl.c. Instead hv_func.h drives the whole
process and perl.c just does what it is told.

Previously the way to control SBOX32 was to use a define with a value
but our perl -V output currently doesnt support that, so this adds some
two new defines PERL_HASH_USE_SBOX32 and PERL_HASH_NO_SBOX32 which map
to the older PERL_HASH_USE_SBOX32_ALSO flag define (integer 1/0).
Both are still supported, this just makes everything more consistent.

This also includes minor doc changes to INSTALL to mention -Accflags
as being the way to set these defines during the Configure process.


  Commit: 00b42049cd2abeb11d2831eec4e351cf053aeb36
      https://github.com/Perl/perl5/commit/00b42049cd2abeb11d2831eec4e351cf053aeb36
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2022-08-09 (Tue, 09 Aug 2022)

  Changed paths:
    M perl.c
    M perl.h

  Log Message:
  -----------
  perl.[ch] - hash related setup affects binary compatibility

Our hash function is compiled into code which chooses to use it,
and is not linked in from the perl executable. That means that
the hash related settings can affect binary compatibility.

I proposed making the hash function linkable, but people said "there was
no need". But that means that the hash function config DOES affect
binary compatibility.


Compare: https://github.com/Perl/perl5/compare/f7e7b4d5e7c8...00b42049cd2a



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