Quoting Matti Picus <[EMAIL PROTECTED]>:

> UT_UCS_strcpy_char instantiates a UT_Mbtowc class instance on each 
> call.  As the default constructor uses values that should remain
> constant 
> throughout the lifetime of the application, creating and destroying the 
> UT_Mbtowc instance is unnecessary.  Especially in a function as
> frequently 
> used as UT_USC_strcpy_char. Here is a patch to make the variable static 
> (created only once).  Could someone review this and (hopefully) check it
> in?
> Thanks,
> Matti

Great catch. Committed.

Dom

Reply via email to