RFR: 8157965 update httpserver logging to use java.lang.System.Logger

Chris Hegarty chris.hegarty at oracle.com
Thu Oct 20 08:20:06 UTC 2016


> On 19 Oct 2016, at 16:29, Daniel Fuchs <daniel.fuchs at oracle.com> wrote:
> 
> Thanks Roger!
> 
> On 19/10/16 16:01, Roger Riggs wrote:
>> Hi Daniel,
>> 
>> looks fine.
>> 
>> (The style differences are disconcerting but are consistent within the
>> file).
> 
> Yes. I preserved the original style though it was hurting
> my eyes ;-) Having a mixed style was much worse.

Looks good Daniel. 

Maybe worth a followup to run these files through a reasonable IDE reformatter.

-Chris.

> best regards,
> 
> -- daniel
> 
>> 
>> Roger
>> 
>> 
>> On 10/19/2016 10:52 AM, Daniel Fuchs wrote:
>>> Hi,
>>> 
>>> Please find below a patch that updates jdk.httpserver to use
>>> System.Logger.
>>> 
>>> As a result jdk.httpserver no longer requires java.logging.
>>> 
>>> webrev: http://cr.openjdk.java.net/~dfuchs/webrev_8157965/webrev.00
>>> issue: https://bugs.openjdk.java.net/browse/JDK-8157965
>>> 
>>> best regards,
>>> 
>>> -- daniel
>> 
> 



More information about the net-dev mailing list