[Bug 30159] New: Regression: Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Bug #: 30159
           Summary: Regression: Font replacement doesn't work
           Product: Wine
           Version: 1.4
          Platform: x86
        OS/Version: Linux
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: gdi32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

LANG=zh_CN.utf8

"WenQuanYi Micro Hei" is installed.

for ubuntu:
$ dpkg -S /usr/share/fonts/truetype/wqy/wqy-microhei.ttc 
ttf-wqy-microhei: /usr/share/fonts/truetype/wqy/wqy-microhei.ttc

The below to lines are added to wine.inf :
--- snip ---
HKCU,Software\Wine\Fonts\Replacements,"SimSun",,"WenQuanYi Micro Hei"
HKCU,Software\Wine\Fonts\Replacements,"PMingLiu",,"WenQuanYi Micro Hei"
--- snip ---

In previous version, wine notepad display Chinese correctly with above
settings,
in latest wine, Chinese display broken in lot's program including notepad input
area.

$ git bisect good
eab02574929df3a5a6e6dcdb26d4d9a629618e24 is the first bad commit
commit eab02574929df3a5a6e6dcdb26d4d9a629618e24
Author: Kusanagi Kouichi 
Date:   Thu Mar 8 22:33:20 2012 +0900

gdi32: Simplify font replacement loading.

:040000 040000 67c115fbf494f8511413b0bddaeb6febe46a1ebf
e9f19ea77acc9a3005d58948e64d604ee8738305 M    dlls

[Bug 30159] Regression: Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Qian Hong  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download

[Bug 30159] Regression: Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Qian Hong  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Dmitry Timoshkov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |regression
            Summary|Regression: Font            |Font replacement doesn't
                   |replacement doesn't work    |work
    Regression SHA1|                            |eab02574929df3a5a6e6dcdb26d
                   |                            |4d9a629618e24

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #1 from Kusanagi Kouichi  2012-03-14 03:45:28 CDT ---
Created attachment 39349
  --> http://bugs.winehq.org/attachment.cgi?id=39349
patch

Does this patch fix the regression?

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #2 from Qian Hong  2012-03-14 05:31:55 CDT ---
(In reply to comment #1)
> Created attachment 39349 [details]
> patch
>
> Does this patch fix the regression?

Hi Kusanagi, thank a lot, it works.

BTW, Bug 29851 still present in latest git with this patch, could you have a
look as well?

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #3 from Kusanagi Kouichi  2012-03-14 07:25:21 CDT ---
patch sent:
http://source.winehq.org/patches/data/84485

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Jerome Leclanche  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch
                 CC|                            |[e-mail]

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #4 from Nikolay Sivov  2012-03-15 23:49:59 CDT ---
Patch committed as
http://source.winehq.org/git/wine.git/commit/6a49adeee6f889913bc960ce3483d83....
Please confirm it's fixed.

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #5 from Qian Hong  2012-03-15 23:51:04 CDT ---
(In reply to comment #4)
> Patch committed as
> http://source.winehq.org/git/wine.git/commit/6a49adeee6f889913bc960ce3483d83....
> Please confirm it's fixed.

Sorry for delay, yes it's fixed. Thanks!

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Nikolay Sivov  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |6a49adeee6f889913bc960ce348
                   |                            |3d83a1d777ee5
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #6 from Nikolay Sivov  2012-03-15 23:52:14 CDT ---
Fixed.

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

--- Comment #7 from Alexandre Julliard  2012-03-16 14:09:12 CDT ---
Closing bugs fixed in 1.5.0.

[Bug 30159] Font replacement doesn't work

http://bugs.winehq.org/show_bug.cgi?id=30159

Alexandre Julliard  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #8 from Alexandre Julliard  2012-03-16 17:11:35 CDT ---
Really closing bugs fixed in 1.5.0.