Hi Alan, Joe
I pushed the revised webrev out (webrev.01). I am going through a directory at time to resolve the lint errors and found a few additional classes to address in com.sun.rowset (as I had not looked at this directory yet) which you will see in the change log. Regards, lance On Apr 11, 2011, at 2:48 PM, Alan Bateman wrote: > Lance Andersen - Oracle wrote: >> : >>> >>> PS: javac was updated recently to emit warnings for unreachable catch >>> clauses and one place that generates this warning is >>> src/share/classes/com/sun/rowset/CachedRowSetImpl.java:947. It would be >>> great to get that one fixed while you are fixing the other warnings. >> >> If you are OK with do it as part of this change, happy to change. > Absolutely, warnings needs to be eliminated. Looks to me that the try/catch > can be completely removed from that method. > > -Alan. Lance Andersen| Principal Member of Technical Staff | +1.781.442.2037 Oracle Java Engineering 1 Network Drive Burlington, MA 01803 lance.ander...@oracle.com
