[Fortress] Code smell...

[Emmanuel Lécharny]

Hi Shawn,

there is one method in CommandLineInterpreter, printPermission, which
contains a line of code that I found fishy :


                LOG.info(type + "   KEY" + ++ctr + " [" + key + "]");

Note the ++ctr here.

That means if we remove the LOG, the ctr counter will not be incremented.

Do you agree that we should move the ++ctr out of this LOG statement ?

Thanks !