Am 12.01.2013 22:56, schrieb Lance Andersen - Oracle:
Hi
This is a review request for 8006139 which adds missing methods to 
SQLInput/Output

The webrev can be found at http://cr.openjdk.java.net/~lancea/8006139/webrev.00/

Hi,

are you paid by code lines ;-)
Additionally you have left out one opportunity
... or is it new code style, having 2 wrong indented closing braces in one line?

I would code:

 811     public String readNString() throws SQLException {
 812         String attrib = (String)getNextAttribute();
 813         lastValueWasNull =attrib == null;
 814return attrib;
 815     }

Additionally I'm wondering, whether getNextAttribute() could be generified.

-Ulf

Reply via email to