Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Marcus Larsson
On 04/08/2016 03:26 PM, Bengt Rutisson wrote: Hi Robbin, On 2016-04-08 14:45, Robbin Ehn wrote: Hi again, I missed LogLevel:Off, it should be: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Thanks Marcus! /Robbin On 04/08/2016 03:41 PM, Marcus Larsson wrote: On 04/08/2016 03:26 PM, Bengt Rutisson wrote: Hi Robbin, On 2016-04-08 14:45, Robbin Ehn wrote: Hi again, I missed LogLevel:Off, it should be: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp ---

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Thanks Bengt! /Robbin On 04/08/2016 03:26 PM, Bengt Rutisson wrote: Hi Robbin, On 2016-04-08 14:45, Robbin Ehn wrote: Hi again, I missed LogLevel:Off, it should be: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Bengt Rutisson
Hi Robbin, On 2016-04-08 14:45, Robbin Ehn wrote: Hi again, I missed LogLevel:Off, it should be: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++ b/src/share/vm/logging/logTagSet.cppFri Apr 08 14:44:33

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Hi again, I missed LogLevel:Off, it should be: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++ b/src/share/vm/logging/logTagSet.cppFri Apr 08 14:44:33 2016 +0200 @@ -98,6 +98,7 @@ const size_t

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Hi all, Updated after reviews: diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++ b/src/share/vm/logging/logTagSet.cppFri Apr 08 14:26:44 2016 +0200 @@ -98,6 +98,7 @@ const size_t vwrite_buffer_size

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Hi Bengt, On 04/08/2016 02:16 PM, Bengt Rutisson wrote: Hi Robbin, On 2016-04-08 14:19, Marcus Larsson wrote: Hi Robbin, On 04/08/2016 01:54 PM, Robbin Ehn wrote: Hi all, Please review, Bug: https://bugs.openjdk.java.net/browse/JDK-8153845 Tested with jprt. diff -r 35cb720769c5

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Bengt Rutisson
Hi Robbin, On 2016-04-08 14:19, Marcus Larsson wrote: Hi Robbin, On 04/08/2016 01:54 PM, Robbin Ehn wrote: Hi all, Please review, Bug: https://bugs.openjdk.java.net/browse/JDK-8153845 Tested with jprt. diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp ---

Re: RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Marcus Larsson
Hi Robbin, On 04/08/2016 01:54 PM, Robbin Ehn wrote: Hi all, Please review, Bug: https://bugs.openjdk.java.net/browse/JDK-8153845 Tested with jprt. diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++

RFR(xs): 8153845: UL log write method missing essential assert

2016-04-08 Thread Robbin Ehn
Hi all, Please review, Bug: https://bugs.openjdk.java.net/browse/JDK-8153845 Tested with jprt. diff -r 35cb720769c5 src/share/vm/logging/logTagSet.cpp --- a/src/share/vm/logging/logTagSet.cppFri Apr 08 08:51:45 2016 +0200 +++ b/src/share/vm/logging/logTagSet.cppFri Apr 08