Maybe.  I've always hated isDebugEnabled flags.  It clutters things.  This loop is really short and small.  And it happened to be where the problem was in my modifications.  The statement inside doesn't execute unless the logging level is at debug.  So, I see this change as extra complexity, without much benefit.  I'd even go for taking it out or commenting it out before I'd put the flag in there.  That's just my preference.  I can see your point of having a needless loop in there.