On Wed, 16 Aug 2023 09:11:34 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
On Wed, 16 Aug 2023 09:11:34 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
On Thu, 10 Aug 2023 11:54:05 GMT, Cristian Vat wrote:
>> Shouldn't the similar change be in `CIBackRef.match` too? The fact current
>> tests do not catch it makes me uneasy: the test coverage seems to be rather
>> low there.
>>
>> We need a regex expert to look at it. @rgiulietti @igraves mig
On Thu, 3 Aug 2023 12:34:19 GMT, Raffaello Giulietti
wrote:
>> Cristian Vat has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> changes and test for CIBackRef
>
> The change looks good.
> However, I'm not a Reviewer.
This looks good to me,
On Fri, 28 Jul 2023 08:40:30 GMT, Aleksey Shipilev wrote:
>> Cristian Vat has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> reduce allocations also for Matcher.usePattern
>
> Shouldn't the similar change be in `CIBackRef.match` too? The fa
On Fri, 28 Jul 2023 08:40:30 GMT, Aleksey Shipilev wrote:
>> Cristian Vat has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> reduce allocations also for Matcher.usePattern
>
> Shouldn't the similar change be in `CIBackRef.match` too? The fa
On Thu, 3 Aug 2023 14:42:45 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are al
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
On Fri, 28 Jul 2023 12:15:21 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
On Fri, 28 Jul 2023 12:15:21 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
On Fri, 28 Jul 2023 08:25:55 GMT, Aleksey Shipilev wrote:
>> Cristian Vat has updated the pull request incrementally with one additional
>> commit since the last revision:
>>
>> reduce allocations also for Matcher.usePattern
>
> src/java.base/share/classes/java/util/regex/Pattern.java line 51
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
On Fri, 28 Jul 2023 09:29:48 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
On Sat, 22 Jul 2023 12:13:10 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
On Sat, 22 Jul 2023 12:13:10 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
On Sat, 22 Jul 2023 12:13:10 GMT, Cristian Vat wrote:
>> Reduces excessive allocation of Matcher.groups array when the original
>> Pattern has no groups or less than 9 groups.
>>
>> Original clamping to 10 possibly due to documented behavior from javadoc:
>> "In this class, \1 through \9 are a
> Reduces excessive allocation of Matcher.groups array when the original
> Pattern has no groups or less than 9 groups.
>
> Original clamping to 10 possibly due to documented behavior from javadoc:
> "In this class, \1 through \9 are always interpreted as back references, "
>
> Only with Matche
Reduces excessive allocation of Matcher.groups array when the original Pattern
has no groups or less than 9 groups.
Original clamping to 10 possibly due to documented behavior from javadoc:
"In this class, \1 through \9 are always interpreted as back references, "
Only with Matcher changes RegE
21 matches
Mail list logo
