review request 7097386: Correct error in Predicate javadoc example
Lance Andersen - Oracle
Lance.Andersen at oracle.com
Mon Sep 16 11:12:09 UTC 2013
Thanks for the input.
On Sep 16, 2013, at 4:58 AM, Aleksey Shipilev wrote:
> On 09/15/2013 10:38 PM, Lance Andersen - Oracle wrote:
>> I added a webrev
>> http://cr.openjdk.java.net/~lancea/7097386/webrev.00/ as it might be
>> a bit easier for this review.
>
> Notes:
> - change C-style "int v[]" declarations to Java-ish "int[] v".
I changed this.
> - catching SQLException should probably return "false" right away?
True, was a missing cut and paste from the code I copied
> - "if" block can float up to the exception handler block, eliminating
> the need for "int value = 0" line.
I left it as is as I try to keep the try/catch for SQLExceptions to just relevant code when applicable.
>
> Otherwise, looks good (not a Reviewer).
>
again, thank you for the feedback.
Changes are at http://cr.openjdk.java.net/~lancea/7097386/webrev.01/
Best
Lance
> Thanks,
> -Aleksey,
-------------- next part --------------
Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037
Oracle Java Engineering
1 Network Drive
Burlington, MA 01803
Lance.Andersen at oracle.com
More information about the core-libs-dev
mailing list