Skip to main content

Please review fix for 6572386 "TestSuite.GeneralPurposeLogger overrides the logger name of the original log record"

1 reply [Last post]
Anonymous

Please review fix for 6572386 "TestSuite.GeneralPurposeLogger overrides
the logger name of the original log record"

The diff is here:
https://jtharness.dev.java.net/source/browse/jtharness/branches/users/er...

The submitter checked jar with proposed fix and confirmed that result
fixes the issue.

Thanks in advance,

Mike

---------------------------------------------------------------------
To unsubscribe, e-mail: interest-unsubscribe@jtharness.dev.java.net
For additional commands, e-mail: interest-help@jtharness.dev.java.net

Reply viewing options

Select your preferred way to display the comments and click "Save settings" to activate your changes.
bkurotsu
Offline
Joined: 2004-12-13
Points: 0

Looks fine, go ahead and merge into trunk.

BTW, the main concern I would have had is the removal of API. But since it is just an override of the Logger method, it is safe.