Re: RFR 7125055: ContentHandler.getContent API changed in error

2011-12-30 Thread Alan Bateman

On 28/12/2011 09:38, chris hegarty wrote:
During some of the warning cleanups the method declaration of the 
public platform API java.net.ContentHandler.getContent(URLConnection, 
Class[]) was incorrectly changed. This CR proposes to revert this 
change fix appropriate subclasses, so as not to re-introduce warnings 
in this area.


http://cr.openjdk.java.net/~chegar/7125055/webrev.00/webrev/

-Chris.
Looks okay to me, would be good to fix the indentation in these methods 
while you are  there.


-Alan.


Re: RFR 7125055: ContentHandler.getContent API changed in error

2011-12-30 Thread Chris Hegarty
Thanks Alan. My editor was not showing these indentation issues. I will fix 
them before pushing.

-Chris

Alan Bateman alan.bate...@oracle.com wrote:

On 28/12/2011 09:38, chris hegarty wrote:
 During some of the warning cleanups the method declaration of the 
 public platform API java.net.ContentHandler.getContent(URLConnection, 
 Class[]) was incorrectly changed. This CR proposes to revert this 
 change fix appropriate subclasses, so as not to re-introduce warnings 
 in this area.

 http://cr.openjdk.java.net/~chegar/7125055/webrev.00/webrev/

 -Chris.
Looks okay to me, would be good to fix the indentation in these methods 
while you are  there.

-Alan.