Request for Review 6882384

Alan Bateman Alan.Bateman at Sun.COM
Tue Sep 22 04:58:22 PDT 2009


Christopher Hegarty - Sun Microsystems Ireland wrote:
>
> 6882384: Update http protocol handler to use PlatformLogger
>
> Webrev:
>   http://cr.openjdk.java.net/~chegar/6882384/webrev.0/webrev/
>
> Change http logging to use the PlatformLogger. We can simply remove 
> the reflective calls to the logging API from HttpCapture as the 
> PlatformLogger will take care of that for us.
>
> -Chris.
Looks good to me (and I see you've fixed a delegation issue in 
PlatformLogger). Minor nit is that the indent seems to be messed up in 
HttpClient line 809.

-Alan.



More information about the net-dev mailing list