Thank you. Review again before submitting a patch to wine-patches, please.
On 02/26/13 00:08, Akihiro Sagawa wrote:
On Sun, 24 Feb 2013 09:55:59 +0900, Byeongsik Jeon wrote:
@@ -3615,6 +3675,10 @@ static void update_font_info(void) } if (!done) FIXME("there is no font defaults for codepages %u,%u\n", ansi_cp, oem_cp);
- /* update locale dependent font association info in registry */
- if (strcmp(buf, cpbuf))
update_font_association_info(ansi_cp);}
static BOOL init_freetype(void)
This condition should be rewritten. buf is not initialized when RegQueryValueExA is failed in line 3542. And strcmp(buf, cpbuf) has already done in line 3544.
I wanted to update in registry when codepages changed, not logpixels. When logpixels has been changed, MS-Windows does not update the font associated charset registry. "memset(buf, 0, sizeof(buf))" has been added.
diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c index 943f584..9d57806 100644 --- a/dlls/gdi32/tests/font.c +++ b/dlls/gdi32/tests/font.c
Why are there no tests against OEM_CHARSET and SYMBOL_CHARSET?
In the current Wine code, if lfCharset == OEM_CHARSET then GetTextCharset() return a wrong value. This problem can be found in the test_oemcharset(). At this point, I wanted to concentrate on the ANSI_CHARSET association problem.