[Bug 14160] New: Surface conversion problem with native d3dx

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

Summary: Surface conversion problem with native d3dx
           Product: Wine
           Version: CVS/GIT
          Platform: Other
               URL: http://www16.big.or.jp/~zun/html/th10dl.html
        OS/Version: other
            Status: UNCONFIRMED
          Severity: normal
          Priority: P2
         Component: -unknown
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]

Created an attachment (id=14422)
 --> (http://bugs.winehq.org/attachment.cgi?id=14422)
wrong output with the native d3dx_31.dll

Touhou 10 has a problem with the displayed image when you pause the game.  The
surface is corrupted (as if the conversion from the original surface was done
improperly).

I know the function used is D3DXLoadSurfaceFromSurface, a simple implementation
with StretchRect gets it right, but not the native library (this also goes for
the older games which embed d3dx8).

[Bug 14160] Surface conversion problem with native d3dx

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

--- Comment #1 from Philip Nilsson   2008-06-28 08:00:32 ---
Created an attachment (id=14423)
 --> (http://bugs.winehq.org/attachment.cgi?id=14423)
right output with a simple implementation of D3DXLoadSurfaceFromSurface

[Bug 14160] Surface conversion problem with native d3dx

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |INVALID

--- Comment #2 from Austin English   2008-06-28 17:05:50 ---
How is this a wine bug if the bug is in the native DLL?

[Bug 14160] Surface conversion problem with native d3dx

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

Philip Nilsson  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
         Resolution|INVALID                     |

--- Comment #3 from Philip Nilsson   2008-06-29 04:45:04 ---
It works on Windows, meaning there is some problem with the functions the
native D3DX calls.

[Bug 14160] Surface conversion problem with native d3dx

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

joaopa  changed:

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

--- Comment #4 from joaopa   2008-06-29 05:03:17 ---
Don't yoou confuse native and build-in dlls?

native is the one provided by windows, built-in these one provided by wine.

Anyway, did you correct the wrong function in wine? If yes, can you send a
patch to wine-patches?

Joaopa

[Bug 14160] Surface conversion problem with native d3dx

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

--- Comment #5 from Philip Nilsson   2008-06-29 06:44:23 ---
> native is the one provided by windows, built-in these one provided by wine.

That's the definition I used.
> Anyway, did you correct the wrong function in wine? If yes, can you send a patch to wine-patches?

It's far from good enough, and that'd just hide this bug (I did some digging
and found the native D3DXLoadSurfaceFromSurface does nothing more than GetDesc
and Lock/Unlock in all cases (filtering, format conversion), the bug must be
earlier).

[Bug 14160] Surface conversion problem with native d3dx

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

--- Comment #6 from Philip Nilsson   2008-06-29 13:49:24 ---
The surface copied from has usage rendertarget and pool default, it also
appears to be the back-buffer since GetContainer returns a SwapChain object.

I don't think StretchRect is supposed to be able to work like this...

[Bug 14160] Surface conversion problem with native d3dx

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

Bryan Donlan  changed:

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

[Bug 14160] Surface conversion problem with native d3dx

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

Philip Nilsson  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED

--- Comment #7 from Philip Nilsson   2009-01-13 17:26:07 ---
Fixed by 9dea6f7de5cf5161825f24b6b33f2e0e15b83ab0: wined3d: Fix glReadPixels
call from read_from_framebuffer.

[Bug 14160] Surface conversion problem with native d3dx

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

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
            Version|CVS/GIT                     |unspecified

[Bug 14160] Surface conversion problem with native d3dx

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

Alexandre Julliard  changed:

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

--- Comment #8 from Alexandre Julliard   2009-01-16 10:38:43 ---
Closing bugs fixed in 1.1.13.