Branch: refs/heads/blead Home: https://github.com/Perl/perl5 Commit: a170805d60a9631c88faf0da744391bfc8f2fe26 https://github.com/Perl/perl5/commit/a170805d60a9631c88faf0da744391bfc8f2fe26 Author: Karl Williamson <k...@cpan.org> Date: 2023-02-10 (Fri, 10 Feb 2023)
Changed paths: M util.c Log Message: ----------- util.c: White-space only Properly indent some preprocessor directives Commit: a683fa5b7e816ae5c10d246c9a0b1f3ea743274b https://github.com/Perl/perl5/commit/a683fa5b7e816ae5c10d246c9a0b1f3ea743274b Author: Karl Williamson <k...@cpan.org> Date: 2023-02-10 (Fri, 10 Feb 2023) Changed paths: M perl.h M perlio.c M util.c Log Message: ----------- snprintf() calls need to have proper radix Calls to libc snprintf() were neglected to be changed when perl was fixed to change the radix character to the proper one based on whether or not 'use locale' is in effect. Perl-level code is unaffected, but core and XS code is. This commit changes to wrap snprintf() calls with the macros designed for the purpose, long used for similar situations elsewhere in the code. Doing this requires the thread context. I achieved this in a few places by a dTHX, instead of assuming a caller would have the context already available, and adding a pTHX_ parameter. I tried doing it the other way, and got a few breakages in our test suite. Formatting already requires significant CPU time, so this addition should just be in the noise This bug was found by new tests that will be added in a future commit. Compare: https://github.com/Perl/perl5/compare/2ba5f4dfa884...a683fa5b7e81