Perhaps, but in that case, it should probably assert instead of breaking out of the loop.  If 'msg' were ever 'null', the current behavior would be to NPE inside of StringUtil.simpleClassName in the code below.