develooper Front page | perl.perl5.changes | Postings from February 2021

[Perl/perl5] d18575: hv_func.h: Move static declaration so it workswit...

From:
Yves Orton via perl5-changes
Date:
February 12, 2021 18:21
Subject:
[Perl/perl5] d18575: hv_func.h: Move static declaration so it workswit...
Message ID:
Perl/perl5/push/refs/heads/blead/2e2ef7-a5d5b4@github.com
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: d18575f18c6ee61ce80492e82cae7361358d570a
      https://github.com/Perl/perl5/commit/d18575f18c6ee61ce80492e82cae7361358d570a
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M hv_func.h

  Log Message:
  -----------
  hv_func.h: Move static declaration so it works with Siphash builds


  Commit: aae087f7cec022be14a17deb95cb2208e16b7891
      https://github.com/Perl/perl5/commit/aae087f7cec022be14a17deb95cb2208e16b7891
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M hv.c

  Log Message:
  -----------
  hv.c: add a guard clause to prevent the number of buckets in a hash from getting too large

This caps it at 1<<28 buckets, eg, ~268M. In theory without a guard clause like
this we could grow to the point of possibly wrapping around in terms of size,
not to mention being ridiculously wasteful of memory at larger sizes.
Even this cap is probably too high. It should probably be something like 1<<24.


  Commit: 6027b190154088fbbcbde08a80c49531e4e4c012
      https://github.com/Perl/perl5/commit/6027b190154088fbbcbde08a80c49531e4e4c012
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M hv_func.h

  Log Message:
  -----------
  [#18555] hv_func.h - Ensure that hash related data is properly aligned

Andrew Fresh reported that the data being unaligned causes issues and
performance degredation under some or all builds. This is an enhacement
of his original patch to handle the 32 bit case, and to make some
related magic numbers more obvious.


  Commit: 19e01a4b5d863a6fbf2322e909618f2ae39e50b3
      https://github.com/Perl/perl5/commit/19e01a4b5d863a6fbf2322e909618f2ae39e50b3
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M ext/Hash-Util/Changes
    M ext/Hash-Util/Util.xs
    M ext/Hash-Util/lib/Hash/Util.pm
    M ext/Hash-Util/t/Util.t

  Log Message:
  -----------
  Hash::Util - Add tests for hash_value($str,$seed) which was broken for a while

Also tweak a sprintf not to warn. This bumps the version to 0.25 and
adds missing (and uniniteresting) Changes file.


  Commit: 4de2690c76c1d4569e6e698f0cf54b7ec8e183b9
      https://github.com/Perl/perl5/commit/4de2690c76c1d4569e6e698f0cf54b7ec8e183b9
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M t/opbasic/arith.t

  Log Message:
  -----------
  opbasic/arith.t - make sure we use the right Config

We were warning about only using Config::Config once. Digging it seems
that the Config module is only loaded under VMS, and the checks on $Config{foo}
were checking an empty hash. This changes them to use $Config::Config{foo} so
that we use the right hash when Config.pm is required in the VMS code


  Commit: 02796811fc1c6df226e84718831dbe2354e4e181
      https://github.com/Perl/perl5/commit/02796811fc1c6df226e84718831dbe2354e4e181
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M ext/PerlIO-encoding/encoding.pm
    M ext/PerlIO-encoding/encoding.xs

  Log Message:
  -----------
  PerlIO-encoding - add parens to silence build warnings


  Commit: a5d5b4db3f3617b82434ee704fc42b49388f3ae0
      https://github.com/Perl/perl5/commit/a5d5b4db3f3617b82434ee704fc42b49388f3ae0
  Author: Yves Orton <demerphq@gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M ext/PerlIO-encoding/encoding.xs

  Log Message:
  -----------
  encoding.xs - silence unused var warnings

in 370c6ab2e9608a94096854c61d976ccf65bb2c13 logic to set PerlIO::encoding::fallback in
encoding.xs was removed, but this var was missed in the cleanup.


Compare: https://github.com/Perl/perl5/compare/2e2ef7683285...a5d5b4db3f36



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