Hi Marco, On Thu, 2011-04-14 at 19:12 +0100, Caolán McNamara wrote: > On Thu, 2011-04-14 at 19:07 +0200, Marco wrote: > > Hi all, > > "Store/Restore font dropdown MRU" is the required hack I worked on. > > I attached the related patch. > > oh cool, one little thing, I wouldn't bother with using the > osl_getThreadTextEncoding() (which is just the system encoding) and > instead just use RTL_TEXTENCODING_UTF8, that way we don't run into > problems munging e.g. CJK font names into "?????" if the system text > encoding is some 8bit foo.
I just pushed your patch after: * Applying Caolan's comment * Adding #if OSL_DEBUG_LEVEL > 1 around the include of cstdio Many thanks for your nice patch and welcome to the LibreOffice hacking community! -- Cédric Bosdonnat LibreOffice hacker http://documentfoundation.org OOo Eclipse Integration developer http://cedric.bosdonnat.free.fr _______________________________________________ LibreOffice mailing list LibreOffice@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/libreoffice
