Branch: refs/heads/blead
  Home:   https://github.com/Perl/perl5
  Commit: 043cdb9a51f856fc6e4eb34f96c740b8b3c9cbcb
      
https://github.com/Perl/perl5/commit/043cdb9a51f856fc6e4eb34f96c740b8b3c9cbcb
  Author: Karl Williamson <[email protected]>
  Date:   2019-12-27 (Fri, 27 Dec 2019)

  Changed paths:
    M regcomp.c

  Log Message:
  -----------
  regcomp.c: Change names of 2 macros

This is in preparation for the next commit, reflecting their new purpose


  Commit: bf18a2468965728c24982b0c89ac0022f9e20b77
      
https://github.com/Perl/perl5/commit/bf18a2468965728c24982b0c89ac0022f9e20b77
  Author: Karl Williamson <[email protected]>
  Date:   2019-12-27 (Fri, 27 Dec 2019)

  Changed paths:
    M regcomp.c
    M t/re/pat.t

  Log Message:
  -----------
  PATCH: GH #17384 out of bounds read in qr//

This turned out to be because there are two versions of the property
name being parsed: 1) the original input; and 2) a canonicalized one
with characters squeeezed out that are usually optional, such as spaces,
dashes and, here, underscores.

The code was conflating the two names, and moving along the squeezed
name based on counts from the unsqueezed one, hence going too far in the
buffer.


  Commit: 4a808c87ccea54be9a4906af92d5d8366df3748b
      
https://github.com/Perl/perl5/commit/4a808c87ccea54be9a4906af92d5d8366df3748b
  Author: Karl Williamson <[email protected]>
  Date:   2019-12-27 (Fri, 27 Dec 2019)

  Changed paths:
    M regcomp.c

  Log Message:
  -----------
  regcomp.c: Use marker for non-user properties

A marker is created for a property that could be both a user-defined
one, and an official Unicode one.  The expansion of these is deferred
until runtime, in case the sub that implements a user-defined property
wasn't known at the time of the pattern compilation.  If at runtime,
there still isn't the proper sub, the property is looked up as
potentially an official Unicode one.

But I forgot to check the marker, which is set only if there is a
potential ambiguity.  If the marker isn't set, there is no need to try
to look it up as an official Unicode property.  I couldn't think of a
test where this fails, but at least it saves effort.


Compare: https://github.com/Perl/perl5/compare/1f9a87c44aa1...4a808c87ccea

Reply via email to