Hi,
Please find the updated webrev after incorporating Paul's comments.
http://cr.openjdk.java.net/~vtheeyarath/8184692/webrev.02/
Also, I have created a csr for this issue
https://bugs.openjdk.java.net/browse/JDK-8201308 .
Regards
Vivek
-----Original Message-----
From: Paul Sandoz
Sent: Friday, April 06, 2018 6:55 AM
To: Vivek Theeyarath <[email protected]>
Cc: Core-Libs-Dev <[email protected]>
Subject: Re: RFR: 8184692: add Pattern.asMatchPredicate
> On Apr 4, 2018, at 10:47 AM, Vivek Theeyarath <[email protected]>
> wrote:
>
> Hi All,
>
> Please review.
>
> Bug: https://bugs.openjdk.java.net/browse/JDK-8184692
>
> Webrev : http://cr.openjdk.java.net/~vtheeyarath/8184692/webrev.00/
>
Like with your other patch, alignment to ~80 chars would be good, as that is
mostly consistent with other code in the same source file.
Let’s not use the word “find" here, so as not to confuse with matcher(s).find().
5833 * @return The predicate which can be used for finding if an input
string matches this pattern.
I suggest:
@return The predicate which can be used for matching an input string against
this pattern
You could also add a @see Matcher#matches
Paul.
>
>
> The related jtreg test was run and the test passed .
>
>
>
> Regards
>
> Vivek
