[Bug 29840] New: gdi32/font.ok test fails: font.c:4208: Test failed: expected non-vertical face for charset 128, got'

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

Bug #: 29840
           Summary: gdi32/font.ok test fails: font.c:4208: Test failed:
                    expected non-vertical face for charset 128, got'
           Product: Wine
           Version: 1.4-rc2
          Platform: x86
               URL: http://source.winehq.org/git/wine.git/commitdiff/28685
                    4fb06aaf68ab43cc175555d927df96bc6ec
        OS/Version: Linux
            Status: NEW
          Keywords: download, regression
          Severity: normal
          Priority: P2
         Component: gdi32
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified
   Regression SHA1: 286854fb06aaf68ab43cc175555d927df96bc6ec

Created attachment 38757
  --> http://bugs.winehq.org/attachment.cgi?id=38757
terminal output

Regression introduced by:
http://source.winehq.org/git/wine.git/commitdiff/286854fb06aaf68ab43cc175555...

fails on my buildbot machine (Debian testing) and my gentoo machine.

I have corefonts installed on both, is some other font needed?

Log attached from make font.ok.

[Bug 29840] gdi32/font.ok test fails: font.c:4208: Test failed: expected non-vertical face for charset 128, got'

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

Sagawa  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |sagawa.aki+winebugs@gmail.c
                   |                            |om

[Bug 29840] gdi32/font.ok test fails: font.c:4208: Test failed: expected non-vertical face for charset 128, got'

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

--- Comment #1 from Sagawa  2012-02-09 10:25:35 CST ---
Hi and sorry for inconvenience.

As wine's RemoveFontResourceEx function is a stub, the font, @WineTestVertical,
which used in test_vertical_font() remains in the font list. Although it is
enumerated in has_vertical_font_proc(), it isn't used for default font
selection.

I'll send the patch later which reorder tests between test_vertical_font() and
test_east_asian_font_selection().

[Bug 29840] gdi32/font.ok test fails: font.c:4208: Test failed: expected non-vertical face for charset 128, got'

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |39f748f083d3c311e288a6ee863
                   |                            |0f42426c830dd
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #2 from Austin English  2012-02-09 15:31:41 CST ---
Thanks for the quick fix!

http://source.winehq.org/git/wine.git/commitdiff/39f748f083d3c311e288a6ee863...

[Bug 29840] gdi32/font.ok test fails: font.c:4208: Test failed: expected non-vertical face for charset 128, got'

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

Alexandre Julliard  changed:

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

--- Comment #3 from Alexandre Julliard  2012-02-11 13:53:01 CST ---
Closing bugs fixed in 1.4-rc3.