Am Montag, 31. Dezember 2007 03:04:02 schrieb Alexander Dorofeyev:
I think there's a bug here. If pal == NULL that implies (because of preceding code) that wine_pal == NULL. So it's setting a NULL wined3d palette to the destination wined3d surface, yet increases reference counter on source palette. I guess it should pass source wined3d palette here, to make any sense. --- dlls/ddraw/texture.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-)
I am not sure about this, but honestly I don't really know much about IDirect3DTexture2::Load is supposed to work. When I ported ddraw.dll to use wined3d I only did routine adjustments to the code to cope with the wined3d objects. I think Lionel Ulmer wrote the original code, maybe he can comment on it?
I think we really need a few unit tests for this method. From the docs, it seems that this method is related to IDirect3DDevice9::LoadTexture, maybe we can map it to WineD3D's loadtexture.