Thanks for the reviews. All callers of EventHelper log methods are
ensuring that isLoggingSecurity() is true before proceeding. I've added
an assert null check in the 4 logger methods to ensure expectations are
in place.
http://cr.openjdk.java.net/~coffeys/webrev.8234466.v5/webrev/
Hope this helps,
Sean.
On 13/01/2020 14:38, Daniel Fuchs wrote:
On 13/01/2020 14:06, Chris Hegarty wrote:
I’m going to ask, since I cannot find the answer myself. Why are some
securityLogger::log invocations guarded with isLoggingSecurity, and
others not? With this change shouldn’t all invocations be guarded,
since it is isLoggingSecurity that assigns securityLogger a value?
Argh! Well spotted chris!
-- daniel