Thanks for reviewing Bengt!
On 01/21/2016 02:05 PM, Bengt Rutisson wrote:
Hi Marcus,
On 2016-01-21 11:42, Marcus Larsson wrote:
Hi,
Please review the following patch to fix an initialization error in
LogTagLevelExpression. If not using the parse() method (as is the
case in LogConfiguration::configure_stdout) instances of the class
would not be properly initialized.
The clear() method has been removed and the necessary initialization
is now done in the constructor instead. An internal VM test has been
added for configure_stdout.
Webrev:
http://cr.openjdk.java.net/~mlarsson/8147348/webrev.00/
Looks good. Thanks for fixing this!
Bengt
Issue:
https://bugs.openjdk.java.net/browse/JDK-8147348
Testing:
New test through JPRT
Thanks,
Marcus