This change leads to incorrect SMAPs of TemplateText nodes with 3 or fewer characters, because the "if (textSize <= 3)" logic isn't mirrored in SmapUtil.java.
Moving the "if (textSize <= 3)" block into the "if (ctxt.getOptions().genStringAsCharArray())" block seems to be a quick way to fix the problem for genStrAsCharArray=false (which, if genStrAsCharArray=true is still experimental, is the only case that really matters), but I haven't followed this thread closely enough to feel confident committing that change.
I think you can go ahead with the change. SMAPs were being generated right, so it should fix the problem IMO.
Remy
--------------------------------------------------------------------- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]