Thanks Staffan!

Marcus

On 01/27/2016 08:56 AM, Staffan Larsen wrote:
Looks good!

Thanks,
/Staffan

On 21 jan. 2016, at 11:42, Marcus Larsson <marcus.lars...@oracle.com> 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/

Issue:
https://bugs.openjdk.java.net/browse/JDK-8147348

Testing:
New test through JPRT

Thanks,
Marcus

Reply via email to