On Wed, Oct 24, 2007 at 10:27:59AM +0200, Jean-Marc Lasgouttes wrote: > Enrico Forestieri <[EMAIL PROTECTED]> writes: > > > Here is a revised patch. I've done some renaming and now don't even > > call anymore the relevant method when a character can be translated > > using the current encoding. > > > +int Paragraph::Pimpl::writeScriptChars(odocstream & os, > > Looks like the variables below could be const. > > > + string script = to_ascii(ltx.substr(1, brace1 - 1));
Not this one, as isKnownScriptChar() expects to be able to modify it when it is empty. -- Enrico
