On 07/12/15 13:55, Paul Sandoz wrote:
On 7 Dec 2015, at 11:18, Chris Hegarty <[email protected]> wrote:
Tagir,
It would seem most useful to update the implementation to
match the current spec.
Yes, it’s an oversight and lacked a test for such case.
Tagir, thanks for finding this. Would it be possible for you to also add a test
to jdk/test/java/util/regex/PatternStreamTest.java?
To that end, your first patch looks
like the most appropriate change, pattern-patch.txt.
The second patch would be more preferable and in line with existing
last-binding implementations.
For me the first patch was "cleaner", but I accept your
point about late-binding. The second patch is as late
as possible.
-Chris.
