[Perl/perl5] 0598e4: perldelta for 6b628002d8

2024-04-23 Thread Tony Cook via perl5-changes 
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 0598e47f3370dd1ec767ac5234803dcb75995dbb
  
https://github.com/Perl/perl5/commit/0598e47f3370dd1ec767ac5234803dcb75995dbb
  Author: Tony Cook 
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
M pod/perldelta.pod

  Log Message:
  ---
  perldelta for 6b628002d8



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

[Perl/perl5] 5b2c91: regcomp_study.c: merged value only used under -DDE...

2024-04-23 Thread Tony Cook via perl5-changes 
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 5b2c91c3c57e411ffed2b23039f769f30abb4d9f
  
https://github.com/Perl/perl5/commit/5b2c91c3c57e411ffed2b23039f769f30abb4d9f
  Author: Tony Cook 
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
M regcomp_study.c

  Log Message:
  ---
  regcomp_study.c: merged value only used under -DDEBUGGING

For a non-debugging build this would warn with clang 17:

regcomp_study.c:1067:9: warning: variable 'merged' set but not used 
[-Wunused-but-set-variable]
 1067 | U32 merged = 0;
  | ^


  Commit: 122500c227656f4f9334ca280614d721dcabacba
  
https://github.com/Perl/perl5/commit/122500c227656f4f9334ca280614d721dcabacba
  Author: Tony Cook 
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
M regcomp_trie.c

  Log Message:
  ---
  regcomp_trie: prevent wordlen value not used warning

This occurs with clang-17, and possibly other versions:

regcomp_trie.c:667:13: warning: variable 'wordlen' set but not used 
[-Wunused-but-set-variable]
  667 | U32 wordlen  = 0; /* required init */
  | ^

This happens because while the first loop in Perl_make_trie calculates
wordlen, mostly via the TRIE_READ_CHAR macro, that calculated value
isn't used.

The later loops do use the value of wordlen via the TRIE_HANDLE_WORD()
macro.

Unfortunately the use in TRIE_READ_CHAR() means we can't remove
this first definition, so suppress the warning.


Compare: https://github.com/Perl/perl5/compare/c37e28edf736...122500c22765

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

[Perl/perl5] c37e28: allow porting/diag.t to detect messages from XSUBs...

2024-04-23 Thread Tony Cook via perl5-changes 
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: c37e28edf73655f7085d46bad8060a781d1bdfed
  
https://github.com/Perl/perl5/commit/c37e28edf73655f7085d46bad8060a781d1bdfed
  Author: Tony Cook 
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
M pod/perldiag.pod
M t/porting/diag.t

  Log Message:
  ---
  allow porting/diag.t to detect messages from XSUBs in class.c

and briefly document the undocumented messages it found.

Note that the "Odd number of arguments" message isn't suppressible,
I do not know if that is by design:

  $ ./perl -Ilib -Mfeature=class -e 'no warnings; class C { field $x:param; } 
C->new("x")'
  Odd number of arguments passed to "C" constructor at -e line 1.

Discovered when porting/diag.t didn't warn about my new message
in the fix for #22159



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

[Perl/perl5] 6b6280: report an error when making an object of an incomp...

2024-04-23 Thread Tony Cook via perl5-changes 
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 6b628002d8c3b4f27498101e8ec682207d5aa466
  
https://github.com/Perl/perl5/commit/6b628002d8c3b4f27498101e8ec682207d5aa466
  Author: Tony Cook 
  Date:   2024-04-24 (Wed, 24 Apr 2024)

  Changed paths:
M class.c
M pod/perldiag.pod
M t/lib/croak/class

  Log Message:
  ---
  report an error when making an object of an incomplete class

instead of asserting or crashing

Fixes #22159



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