On 11/09/2014 09:43, Chris Hegarty wrote:
On 10 Sep 2014, at 16:04, Claes Redestad <claes.redes...@oracle.com> wrote:

On 09/10/2014 04:38 PM, mark.reinh...@oracle.com wrote:
New webrev: http://cr.openjdk.java.net/~redestad/8057936/webrev.4/
Looks fine, but when an exception declaration is on its own line then
the opening brace of the method should be on its own line too, as in
the original:

     protected Class<?> findClass(final String name)
         throws ClassNotFoundException
     {
Original had weird 5 space indentation though:

http://cr.openjdk.java.net/~redestad/8057936/webrev.6/
This latest version looks good.

I agree and much more readable since the API adjustment.

-Alan

Reply via email to