Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: df1cfef7b22a161bc54afb2ae0ac65a96905986f
      
https://github.com/Perl/perl5/commit/df1cfef7b22a161bc54afb2ae0ac65a96905986f
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M perl.h

  Log Message:
  -----------
  perl.h: Comments/white space only

Clarify some comments.


  Commit: bf4a950e7992432fc85bce07e901b7e3a0cc1355
      
https://github.com/Perl/perl5/commit/bf4a950e7992432fc85bce07e901b7e3a0cc1355
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c

  Log Message:
  -----------
  grok_bin_oct_hex: Shortcut leading zeros

I noticed that leading zeros are quite common for octal and hex
constants.  This code is structured for speed, with a partially unrolled
loop structured so that it is impossible to overflow the unrolled part.

If we get to the end of the unrolled portion, and the accumulated value
is still zero, it's because there have been only leading zeroes so far,
and instead of dropping into the loop, we can re-enter the unrolled part
without having to consider the possibility of overflowing.  This allows
the next chunk of digits to be processed without branching.


  Commit: 854ed42b66e322af93078734499a238a2cd26171
      
https://github.com/Perl/perl5/commit/854ed42b66e322af93078734499a238a2cd26171
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c
    M perl.h

  Log Message:
  -----------
  grok_bin_oct_hex: Simplify flag

It's easier to understand and code if each flag name is for a single
flag.


  Commit: 112a9c3e4636e6986be7c704bf14471ba2c7d5e2
      
https://github.com/Perl/perl5/commit/112a9c3e4636e6986be7c704bf14471ba2c7d5e2
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c
    M perl.h
    M regcomp.c

  Log Message:
  -----------
  grok_bin_oct_hex: Change name of internal flag

This clarifies its meaning


  Commit: d4652b085e7b978389c249b8c5b01c42a9c469df
      
https://github.com/Perl/perl5/commit/d4652b085e7b978389c249b8c5b01c42a9c469df
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c

  Log Message:
  -----------
  grok_bin_oct_hex: fix broken return flags

Apparently no one has tried to use these before.  These flags are to
suppress the display of certain warnings, but to instead return that the
suppression happened in output flags.  The output flags were not getting
set.

I'm not adding a separate test, because a future commit will cause this
feature to be used regularly.


  Commit: 2a14b8fc1c56043dfd47f3380225aee90d048a26
      
https://github.com/Perl/perl5/commit/2a14b8fc1c56043dfd47f3380225aee90d048a26
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c

  Log Message:
  -----------
  grok_bin_oct_hex: Avoid two recalculations

We already have this value in a variable.


  Commit: f546c3400ccaccf0ce95500aaac58a50f4fced71
      
https://github.com/Perl/perl5/commit/f546c3400ccaccf0ce95500aaac58a50f4fced71
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

  Changed paths:
    M numeric.c

  Log Message:
  -----------
  grok_bin_oct_hex: Improve overflow detection

The new scheme is is what toke.c already uses for the same purpose, and
I think is cleaner


  Commit: f6e77aca920229d4fdfc933b633d6656841b2d52
      
https://github.com/Perl/perl5/commit/f6e77aca920229d4fdfc933b633d6656841b2d52
  Author: Karl Williamson <[email protected]>
  Date:   2025-11-12 (Wed, 12 Nov 2025)

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

  Log Message:
  -----------
  grok_bin_oct_hex: Make a parameter const

This prevents a future maintainer from accidentally changing this
pointer inside the function, where its original value is needed at the
end


Compare: https://github.com/Perl/perl5/compare/a5e97511521a...f6e77aca9202

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

Reply via email to