Re: PING Re: RFR(s): 8150026: Add the ability to log with variable log level

2016-03-03 Thread Bengt Rutisson


Hi Robbin,

On 2016-03-03 07:41, Robbin Ehn wrote:

Hi,

Could someone else also please review this?

On 02/29/2016 11:22 AM, Marcus Larsson wrote:

Hi Robbin,

On 02/18/2016 08:37 PM, Robbin Ehn wrote:

Hi, please review.

This adds a write method to the UL log which takes the level as
parameter.

JBS: https://bugs.openjdk.java.net/browse/JDK-8150026
Webrev: http://cr.openjdk.java.net/~mlarsson/rehn/8150026/


Looks good, thanks for fixing this.


Looks good to me too.

Bengt



Marcus


Thanks Marcus!

/R





Tested ok with rbt hotspot_all.

Thanks!

/Robbin






Re: PING Re: RFR(s): 8150026: Add the ability to log with variable log level

2016-03-03 Thread Robbin Ehn

Thanks Bengt!

/Robbin

On 03/03/2016 09:35 AM, Bengt Rutisson wrote:


Hi Robbin,

On 2016-03-03 07:41, Robbin Ehn wrote:

Hi,

Could someone else also please review this?

On 02/29/2016 11:22 AM, Marcus Larsson wrote:

Hi Robbin,

On 02/18/2016 08:37 PM, Robbin Ehn wrote:

Hi, please review.

This adds a write method to the UL log which takes the level as
parameter.

JBS: https://bugs.openjdk.java.net/browse/JDK-8150026
Webrev: http://cr.openjdk.java.net/~mlarsson/rehn/8150026/


Looks good, thanks for fixing this.


Looks good to me too.

Bengt



Marcus


Thanks Marcus!

/R





Tested ok with rbt hotspot_all.

Thanks!

/Robbin