Request for Review: 6894169

View: New views

2 Messages — Rating Filter: Alert me

Request for Review: 6894169

by Christopher Hegarty -Sun Microsystems Ireland :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

CR 6894169: Move HttpLogFormatter into a separate package

Webrev:
http://cr.openjdk.java.net/~chegar/6894169/webrev.0/webrev/

Moving HttpLogFormatter out of sun.net.www.protocol.http and into
sun.net.www.protocol.http.logging will simplify the modularization
effort by making it easier to define module definitions (by package name
as opposed to a regular/glob expression). This formatter will not be in
the base package.

-Chris.

Re: Request for Review: 6894169

by Alan Bateman :: Rate this Message:

Reply to Author | View Threaded | Show Only this Message

Christopher Hegarty -Sun Microsystems Ireland wrote:

> CR 6894169: Move HttpLogFormatter into a separate package
>
> Webrev:
> http://cr.openjdk.java.net/~chegar/6894169/webrev.0/webrev/
>
> Moving HttpLogFormatter out of sun.net.www.protocol.http and into
> sun.net.www.protocol.http.logging will simplify the modularization
> effort by making it easier to define module definitions (by package
> name as opposed to a regular/glob expression). This formatter will not
> be in the base package.
>
> -Chris.

Looks good to me.

-Alan