This supposes that the previous fix (6466752) is approved
Looks fine. A style comment though...
In the if statement, the "if" is checking to see if we are in verbose mode. You added an else to make sure that i18n is created if we are not (due to decision at top of method). My comment is that the else action is not really related to the verbose setting, logically speaking.
So perhaps you would consider changing it to:
// print message
if (i18n == null)
i18n = ...
I think it makes a bit more logically sense this way, when reading it. You could argue that it requires an additional conditional evaluation than the original, but the performance effect of that is pretty insignificant. :)
Either way you decide, the fix is fine.
Yes, you're right, logic in my case was not very clear. Your style is better, I used it. Thanks.
Your use of this web site or any of its content or software indicates your agreement to be bound by these Terms of Participation.
Copyright © 2014, Oracle and/or its affiliates. All rights reserved. Oracle and Java are registered trademarks of Oracle and/or its affiliates. Other names may be trademarks of their respective owners.