RFR: JDK-8023647, , " abc1c " .matches( " (\\w)+1\\1 " )) returns false

[Alan Bateman]

On 26/08/2013 21:54, Xueming Shen wrote:
> Hi,
>
> Please help review the proposed change for 8023647:
>
> http://cr.openjdk.java.net/~sherman/8023647/webrev
>
> It appears group index updating in situation of GroupCurly backing off
> is incorrect from day one (the bug is reproducible back to jdk5). The
> current implementation does not back off the index to a previous match,
> but the one is being back off.
>
> The ln#4459 is updating a local variable before return, which is not
> necessary, I removed it with this change.
This looks okay to me, I just wonder if there we need other tests here 
to be confident that there aren't other issues.

-Alan