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