hg: jdk7/tl/jdk: 6843995: RowSet 1.1 updates

Alan Bateman Alan.Bateman at oracle.com
Thu Nov 11 09:59:48 UTC 2010


Lance Andersen - Oracle wrote:
> Hi Remi (and team),
>
> I made changes to SyncFactory and one other class for a similar error. 
>  Also cleaned up a couple of other minor issues in these classes.
>
> The webrev can be found at http://cr.openjdk.java.net/~lancea/6982530/ 
> <http://cr.openjdk.java.net/%7Elancea/6982530/>
>
> Thank you for catching the error.
>
> Regards
> Lance
I've looked through the updated webrev, and I don't see anything 
obviously wrong. I assume getJdbcRowSetResourceBundle could be a little 
bit more efficient so that it only has to read jpResBundle twice rather 
than 3 times during initialization but it's probably not an issue. You 
know this already, but some of this code could do with more clean-up. 
The formatting and code style is a bit inconsistent in places and of 
course there are several unused static fields that someone else 
mentioned. Doing this with a different CR make sense to me.

-Alan.



More information about the core-libs-dev mailing list