Re: RFR(xs): 8148053: Remove unused log tags

2016-01-25 Thread Robbin Ehn
Thanks David! /Robbin On 01/25/2016 02:51 AM, David Holmes wrote: Hi Robbin, Looks good to me too! But as we all play with UL perhaps a broader review/heads-up on hotspot-dev (now cc'd) in case any one is planning on using those. I almost used "rt" in a change I'm working on, but have added

Re: RFR(xs): 8148053: Remove unused log tags

2016-01-24 Thread David Holmes
Hi Robbin, Looks good to me too! But as we all play with UL perhaps a broader review/heads-up on hotspot-dev (now cc'd) in case any one is planning on using those. I almost used "rt" in a change I'm working on, but have added "os" instead. Thanks, David On 23/01/2016 1:00 AM, Robbin Ehn

RFR(xs): 8148053: Remove unused log tags

2016-01-22 Thread Robbin Ehn
Hi, Please review, this removes unused tags in UL. Testing: Passed RBT build Bug: https://bugs.openjdk.java.net/browse/JDK-8148053 Webrev: http://cr.openjdk.java.net/~mlarsson/rehn/8148053/ Thanks and thanks Marcus for hosting! /Robbin

Re: RFR(xs): 8148053: Remove unused log tags

2016-01-22 Thread Staffan Larsen
Looks good! Thanks, /Staffan > On 22 jan. 2016, at 16:00, Robbin Ehn wrote: > > Hi, > > Please review, this removes unused tags in UL. > > Testing: > Passed RBT build > > Bug: https://bugs.openjdk.java.net/browse/JDK-8148053 > Webrev:

Re: RFR(xs): 8148053: Remove unused log tags

2016-01-22 Thread Marcus Larsson
Hi Robbin, On 01/22/2016 04:00 PM, Robbin Ehn wrote: Hi, Please review, this removes unused tags in UL. Testing: Passed RBT build Bug: https://bugs.openjdk.java.net/browse/JDK-8148053 Webrev: http://cr.openjdk.java.net/~mlarsson/rehn/8148053/ Looks good to me! Marcus Thanks and thanks