Groups | Search | Server Info | Keyboard shortcuts | Login | Register [http] [https] [nntp] [nntps]


Groups > perl.perl5.changes > #34307 > unrolled thread

[Perl/perl5] b2b5c0: S_scan_const: abort compilation after \N{} errors

Started byperl5-changes@perl.org (Richard Leach via perl5-changes)
First post2026-01-29 14:16 -0800
Last post2026-01-29 14:16 -0800
Articles 1 — 1 participant

Back to article view | Back to perl.perl5.changes


Contents

  [Perl/perl5] b2b5c0: S_scan_const: abort compilation after \N{} errors perl5-changes@perl.org (Richard Leach via perl5-changes) - 2026-01-29 14:16 -0800

#34307 — [Perl/perl5] b2b5c0: S_scan_const: abort compilation after \N{} errors

Fromperl5-changes@perl.org (Richard Leach via perl5-changes)
Date2026-01-29 14:16 -0800
Subject[Perl/perl5] b2b5c0: S_scan_const: abort compilation after \N{} errors
Message-ID<Perl/perl5/push/refs/heads/blead/a43dbe-b2b5c0@github.com>
  Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: b2b5c0b5fc4c0876397298bfa6459ec385586e63
      https://github.com/Perl/perl5/commit/b2b5c0b5fc4c0876397298bfa6459ec385586e63
  Author: Richard Leach <richardleach@users.noreply.github.com>
  Date:   2026-01-29 (Thu, 29 Jan 2026)

  Changed paths:
    M t/lib/croak/toke
    M toke.c

  Log Message:
  -----------
  S_scan_const: abort compilation after \N{} errors

Upon encountering errors in parsing `\N{}` sequences, the parser used to
try to continue parsing for a bit before exiting. However, these errors
are - under certain circumstances - associated with problems with the
savestack being incorrectly adjusted.

GH #16930 is an example of this where:
* `PL_comppad_name` points to one struct during allocation of pad slots.
* Savestack activity causes `PL_comppad_name` to point somewhere else.
* The peephole optimiser is called, but needs `PL_comppad_name` to point
to the first struct to match up with the pad allocations.

With this commit, errors in parsing `\N{}` sequences are immediately fatal.



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

[toc] | [standalone]


Back to top | Article view | perl.perl5.changes


csiph-web