On Tue, Mar 07, 2006 at 09:10:12PM +0900, Byeong-Sik Jeon wrote:
> ChangeLog:
> init_system_links support .ttc file.
> add "Tahoma" registry value for CJK font link.
> find_face_from_filename support .ttc file
>
I think you could spilt out the 'add Tahoma registry value' bit from
this patch to make it a little smaller.
> --- 000/dlls/gdi/freetype.c 2006-03-07 19:09:53.000000000 +0900
> +++ 001/dlls/gdi/freetype.c 2006-03-07 20:28:14.000000000 +0900
> @@ -909,14 +935,16 @@
> SYSTEM_LINKS *font_link, *system_font_link;
> CHILD_FONT *child_font;
> static const WCHAR Tahoma[] = {'T','a','h','o','m','a',0};
> - static const WCHAR tahoma_ttf[] = {'t','a','h','o','m','a','.','t','t','f',0};
> - static const WCHAR System[] = {'S','y','s','t','e','m',0};
> + static const char Tahoma_ttf[] = {'t','a','h','o','m','a','.','t','t','f',0};
Why don't you just use "tahoma.ttf" in the code where you call
find_face_from_filename, there's no point in initializing a char
string like this.
> @@ -990,15 +1046,15 @@
> that Tahoma has */
>
> system_font_link = HeapAlloc(GetProcessHeap(), 0, sizeof(*system_font_link));
> - system_font_link->font_name = strdupW(System);
> + system_font_link->font_name = strdupW(defSystem);
Please don't use defSystem here. That's supposed to be a fallback for
System. Please leave it as it was.
--
Huw Davies
huw(a)codeweavers.com