RFR 8136496 Connection.begin/endRequest
Roger Riggs
Roger.Riggs at Oracle.com
Thu Nov 5 15:00:10 UTC 2015
Hi Lance,
Looks fine except for a few editorial fixes.
Editorial:
- Update copyrights to 2015.
src/java.sql/share/classes/java/sql/Connection.java:
- about 1515: "." on a separate line.
- several places: missing "." after "no-op"
- later: missing "." after "{@code beginRequest} is a no-op"
src/java.sql/share/classes/javax/sql/PooledConnection.java
- about line 75: the <UL> should be lower case to match </ul>
- you could fix the <P> -> <p> in the line before the edit.
[ There other editorial inconsistencies and indenting in the file; but
maybe that's a different review.]
Roger
On 11/2/2015 12:37 PM, Lance Andersen wrote:
> Hi,
>
> Need a reviewer for 8136496. The CCC has been approved.
>
> The webrev can be found at http://cr.openjdk.java.net/~lancea/8136496/webrev.00/
>
> Best
> Lance
>
>
> 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