wine-commits September 2010

wine-commits@winehq.org

2 participants
852 discussions

Vincent Povirk : windowscodecs: Add tests for IWICBitmapDecoderInfo_GetMimeTypes.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 2d2a6526d88ca1575259b23f3561ebe5d94acc2f URL: http://source.winehq.org/git/wine.git/?a=commit;h=2d2a6526d88ca1575259b23f3… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Sun Sep 5 21:48:31 2010 -0500 windowscodecs: Add tests for IWICBitmapDecoderInfo_GetMimeTypes. --- dlls/windowscodecs/tests/Makefile.in | 1 + dlls/windowscodecs/tests/info.c | 95 ++++++++++++++++++++++++++++++++++ 2 files changed, 96 insertions(+), 0 deletions(-) diff --git a/dlls/windowscodecs/tests/Makefile.in b/dlls/windowscodecs/tests/Makefile.in index d78da2e..e274d3c 100644 --- a/dlls/windowscodecs/tests/Makefile.in +++ b/dlls/windowscodecs/tests/Makefile.in @@ -8,6 +8,7 @@ IMPORTS = ole32 windowscodecs C_SRCS = \ bmpformat.c \ converter.c \ + info.c \ palette.c \ stream.c diff --git a/dlls/windowscodecs/tests/info.c b/dlls/windowscodecs/tests/info.c new file mode 100644 index 0000000..3e5b26a --- /dev/null +++ b/dlls/windowscodecs/tests/info.c @@ -0,0 +1,95 @@ +/* + * Copyright 2009 Vincent Povirk for CodeWeavers + * + * This library is free software; you can redistribute it and/or + * modify it under the terms of the GNU Lesser General Public + * License as published by the Free Software Foundation; either + * version 2.1 of the License, or (at your option) any later version. + * + * This library is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this library; if not, write to the Free Software + * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA + */ + +#include <stdarg.h> +#include <math.h> + +#define COBJMACROS + +#include "windef.h" +#include "objbase.h" +#include "wincodec.h" +#include "wine/test.h" + +static void test_decoder_info(void) +{ + IWICImagingFactory *factory; + IWICComponentInfo *info; + IWICBitmapDecoderInfo *decoder_info; + HRESULT hr; + ULONG len; + WCHAR value[256]; + const WCHAR expected_mimetype[] = {'i','m','a','g','e','/','b','m','p',0}; + + hr = CoCreateInstance(&CLSID_WICImagingFactory, NULL, CLSCTX_INPROC_SERVER, + &IID_IWICImagingFactory, (void**)&factory); + ok(hr == S_OK, "CoCreateInstance failed, hr=%x\n", hr); + if (FAILED(hr)) return; + + hr = IWICImagingFactory_CreateComponentInfo(factory, &CLSID_WICBmpDecoder, &info); + ok(hr == S_OK, "CreateComponentInfo failed, hr=%x\n", hr); + + hr = IWICComponentInfo_QueryInterface(info, &IID_IWICBitmapDecoderInfo, (void**)&decoder_info); + ok(hr == S_OK, "QueryInterface failed, hr=%x\n", hr); + +todo_wine { + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, 0, NULL, NULL); + ok(hr == E_INVALIDARG, "GetMimeType failed, hr=%x\n", hr); + + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, 1, NULL, &len); + ok(hr == E_INVALIDARG, "GetMimeType failed, hr=%x\n", hr); + ok(len == lstrlenW(expected_mimetype)+1, "GetMimeType returned wrong len %i\n", len); + + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, len, value, NULL); + ok(hr == E_INVALIDARG, "GetMimeType failed, hr=%x\n", hr); + + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, 0, NULL, &len); + ok(hr == S_OK, "GetMimeType failed, hr=%x\n", hr); + ok(len == lstrlenW(expected_mimetype)+1, "GetMimeType returned wrong len %i\n", len); + + value[0] = 0; + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, len, value, &len); + ok(hr == S_OK, "GetMimeType failed, hr=%x\n", hr); + ok(lstrcmpW(value, expected_mimetype) == 0, "GetMimeType returned wrong value %s\n", wine_dbgstr_w(value)); + ok(len == lstrlenW(expected_mimetype)+1, "GetMimeType returned wrong len %i\n", len); + + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, 1, value, &len); + ok(hr == WINCODEC_ERR_INSUFFICIENTBUFFER, "GetMimeType failed, hr=%x\n", hr); + ok(len == lstrlenW(expected_mimetype)+1, "GetMimeType returned wrong len %i\n", len); + + hr = IWICBitmapDecoderInfo_GetMimeTypes(decoder_info, 256, value, &len); + ok(hr == S_OK, "GetMimeType failed, hr=%x\n", hr); + ok(lstrcmpW(value, expected_mimetype) == 0, "GetMimeType returned wrong value %s\n", wine_dbgstr_w(value)); + ok(len == lstrlenW(expected_mimetype)+1, "GetMimeType returned wrong len %i\n", len); +} + + IWICBitmapDecoderInfo_Release(decoder_info); + + IWICComponentInfo_Release(info); + + IWICImagingFactory_Release(factory); +} + +START_TEST(info) +{ + CoInitializeEx(NULL, COINIT_APARTMENTTHREADED); + + test_decoder_info(); + + CoUninitialize(); +}

1 0

Vincent Povirk : windowscodecs: Register builtin encoders.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: ad2895d10b62cf756a13ab4336d32c301c90c796 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ad2895d10b62cf756a13ab433… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Sun Sep 5 21:24:53 2010 -0500 windowscodecs: Register builtin encoders. --- dlls/windowscodecs/regsvr.c | 248 ++++++++++++++++++++++++++++++++ dlls/windowscodecs/wincodecs_private.h | 2 + 2 files changed, 250 insertions(+), 0 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ad2895d10b62cf756a13a…

1 0

Vincent Povirk : windowscodecs: Enumerate encoders.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: d701e8fab38643fe4344971adaab033e5ffcfe2e URL: http://source.winehq.org/git/wine.git/?a=commit;h=d701e8fab38643fe4344971ad… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Sun Sep 5 21:13:20 2010 -0500 windowscodecs: Enumerate encoders. --- dlls/windowscodecs/info.c | 255 +++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 255 insertions(+), 0 deletions(-) diff --git a/dlls/windowscodecs/info.c b/dlls/windowscodecs/info.c index f790dbc..373699d 100644 --- a/dlls/windowscodecs/info.c +++ b/dlls/windowscodecs/info.c @@ -467,6 +467,260 @@ static HRESULT BitmapDecoderInfo_Constructor(HKEY classkey, REFCLSID clsid, IWIC } typedef struct { + const IWICBitmapEncoderInfoVtbl *lpIWICBitmapEncoderInfoVtbl; + LONG ref; + HKEY classkey; + CLSID clsid; +} BitmapEncoderInfo; + +static HRESULT WINAPI BitmapEncoderInfo_QueryInterface(IWICBitmapEncoderInfo *iface, REFIID iid, + void **ppv) +{ + BitmapEncoderInfo *This = (BitmapEncoderInfo*)iface; + TRACE("(%p,%s,%p)\n", iface, debugstr_guid(iid), ppv); + + if (!ppv) return E_INVALIDARG; + + if (IsEqualIID(&IID_IUnknown, iid) || + IsEqualIID(&IID_IWICComponentInfo, iid) || + IsEqualIID(&IID_IWICBitmapCodecInfo, iid) || + IsEqualIID(&IID_IWICBitmapEncoderInfo ,iid)) + { + *ppv = This; + } + else + { + *ppv = NULL; + return E_NOINTERFACE; + } + + IUnknown_AddRef((IUnknown*)*ppv); + return S_OK; +} + +static ULONG WINAPI BitmapEncoderInfo_AddRef(IWICBitmapEncoderInfo *iface) +{ + BitmapEncoderInfo *This = (BitmapEncoderInfo*)iface; + ULONG ref = InterlockedIncrement(&This->ref); + + TRACE("(%p) refcount=%u\n", iface, ref); + + return ref; +} + +static ULONG WINAPI BitmapEncoderInfo_Release(IWICBitmapEncoderInfo *iface) +{ + BitmapEncoderInfo *This = (BitmapEncoderInfo*)iface; + ULONG ref = InterlockedDecrement(&This->ref); + + TRACE("(%p) refcount=%u\n", iface, ref); + + if (ref == 0) + { + RegCloseKey(This->classkey); + HeapFree(GetProcessHeap(), 0, This); + } + + return ref; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetComponentType(IWICBitmapEncoderInfo *iface, + WICComponentType *pType) +{ + TRACE("(%p,%p)\n", iface, pType); + *pType = WICEncoder; + return S_OK; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetCLSID(IWICBitmapEncoderInfo *iface, CLSID *pclsid) +{ + FIXME("(%p,%p): stub\n", iface, pclsid); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetSigningStatus(IWICBitmapEncoderInfo *iface, DWORD *pStatus) +{ + FIXME("(%p,%p): stub\n", iface, pStatus); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetAuthor(IWICBitmapEncoderInfo *iface, UINT cchAuthor, + WCHAR *wzAuthor, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchAuthor, wzAuthor, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetVendorGUID(IWICBitmapEncoderInfo *iface, GUID *pguidVendor) +{ + FIXME("(%p,%p): stub\n", iface, pguidVendor); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetVersion(IWICBitmapEncoderInfo *iface, UINT cchVersion, + WCHAR *wzVersion, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchVersion, wzVersion, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetSpecVersion(IWICBitmapEncoderInfo *iface, UINT cchSpecVersion, + WCHAR *wzSpecVersion, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchSpecVersion, wzSpecVersion, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetFriendlyName(IWICBitmapEncoderInfo *iface, UINT cchFriendlyName, + WCHAR *wzFriendlyName, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchFriendlyName, wzFriendlyName, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetContainerFormat(IWICBitmapEncoderInfo *iface, + GUID *pguidContainerFormat) +{ + FIXME("(%p,%p): stub\n", iface, pguidContainerFormat); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetPixelFormats(IWICBitmapEncoderInfo *iface, + UINT cFormats, GUID *pguidPixelFormats, UINT *pcActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cFormats, pguidPixelFormats, pcActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetColorManagementVersion(IWICBitmapEncoderInfo *iface, + UINT cchColorManagementVersion, WCHAR *wzColorManagementVersion, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchColorManagementVersion, wzColorManagementVersion, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetDeviceManufacturer(IWICBitmapEncoderInfo *iface, + UINT cchDeviceManufacturer, WCHAR *wzDeviceManufacturer, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchDeviceManufacturer, wzDeviceManufacturer, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetDeviceModels(IWICBitmapEncoderInfo *iface, + UINT cchDeviceModels, WCHAR *wzDeviceModels, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchDeviceModels, wzDeviceModels, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetMimeTypes(IWICBitmapEncoderInfo *iface, + UINT cchMimeTypes, WCHAR *wzMimeTypes, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchMimeTypes, wzMimeTypes, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_GetFileExtensions(IWICBitmapEncoderInfo *iface, + UINT cchFileExtensions, WCHAR *wzFileExtensions, UINT *pcchActual) +{ + FIXME("(%p,%u,%p,%p): stub\n", iface, cchFileExtensions, wzFileExtensions, pcchActual); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_DoesSupportAnimation(IWICBitmapEncoderInfo *iface, + BOOL *pfSupportAnimation) +{ + FIXME("(%p,%p): stub\n", iface, pfSupportAnimation); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_DoesSupportChromaKey(IWICBitmapEncoderInfo *iface, + BOOL *pfSupportChromaKey) +{ + FIXME("(%p,%p): stub\n", iface, pfSupportChromaKey); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_DoesSupportLossless(IWICBitmapEncoderInfo *iface, + BOOL *pfSupportLossless) +{ + FIXME("(%p,%p): stub\n", iface, pfSupportLossless); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_DoesSupportMultiframe(IWICBitmapEncoderInfo *iface, + BOOL *pfSupportMultiframe) +{ + FIXME("(%p,%p): stub\n", iface, pfSupportMultiframe); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_MatchesMimeType(IWICBitmapEncoderInfo *iface, + LPCWSTR wzMimeType, BOOL *pfMatches) +{ + FIXME("(%p,%s,%p): stub\n", iface, debugstr_w(wzMimeType), pfMatches); + return E_NOTIMPL; +} + +static HRESULT WINAPI BitmapEncoderInfo_CreateInstance(IWICBitmapEncoderInfo *iface, + IWICBitmapEncoder **ppIBitmapEncoder) +{ + BitmapEncoderInfo *This = (BitmapEncoderInfo*)iface; + + TRACE("(%p,%p)\n", iface, ppIBitmapEncoder); + + return CoCreateInstance(&This->clsid, NULL, CLSCTX_INPROC_SERVER, + &IID_IWICBitmapEncoder, (void**)ppIBitmapEncoder); +} + +static const IWICBitmapEncoderInfoVtbl BitmapEncoderInfo_Vtbl = { + BitmapEncoderInfo_QueryInterface, + BitmapEncoderInfo_AddRef, + BitmapEncoderInfo_Release, + BitmapEncoderInfo_GetComponentType, + BitmapEncoderInfo_GetCLSID, + BitmapEncoderInfo_GetSigningStatus, + BitmapEncoderInfo_GetAuthor, + BitmapEncoderInfo_GetVendorGUID, + BitmapEncoderInfo_GetVersion, + BitmapEncoderInfo_GetSpecVersion, + BitmapEncoderInfo_GetFriendlyName, + BitmapEncoderInfo_GetContainerFormat, + BitmapEncoderInfo_GetPixelFormats, + BitmapEncoderInfo_GetColorManagementVersion, + BitmapEncoderInfo_GetDeviceManufacturer, + BitmapEncoderInfo_GetDeviceModels, + BitmapEncoderInfo_GetMimeTypes, + BitmapEncoderInfo_GetFileExtensions, + BitmapEncoderInfo_DoesSupportAnimation, + BitmapEncoderInfo_DoesSupportChromaKey, + BitmapEncoderInfo_DoesSupportLossless, + BitmapEncoderInfo_DoesSupportMultiframe, + BitmapEncoderInfo_MatchesMimeType, + BitmapEncoderInfo_CreateInstance +}; + +static HRESULT BitmapEncoderInfo_Constructor(HKEY classkey, REFCLSID clsid, IWICComponentInfo **ppIInfo) +{ + BitmapEncoderInfo *This; + + This = HeapAlloc(GetProcessHeap(), 0, sizeof(BitmapEncoderInfo)); + if (!This) + { + RegCloseKey(classkey); + return E_OUTOFMEMORY; + } + + This->lpIWICBitmapEncoderInfoVtbl = &BitmapEncoderInfo_Vtbl; + This->ref = 1; + This->classkey = classkey; + memcpy(&This->clsid, clsid, sizeof(CLSID)); + + *ppIInfo = (IWICComponentInfo*)This; + return S_OK; +} + +typedef struct { const IWICFormatConverterInfoVtbl *lpIWICFormatConverterInfoVtbl; LONG ref; HKEY classkey; @@ -662,6 +916,7 @@ struct category { static const struct category categories[] = { {WICDecoder, &CATID_WICBitmapDecoders, BitmapDecoderInfo_Constructor}, + {WICEncoder, &CATID_WICBitmapEncoders, BitmapEncoderInfo_Constructor}, {WICPixelFormatConverter, &CATID_WICFormatConverters, FormatConverterInfo_Constructor}, {0} };

1 0

Henri Verbeet : d3d8: An invalid shader version isn't an internal error.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 95098f1b2a08ea7af895b77cea3cdcaf75869f26 URL: http://source.winehq.org/git/wine.git/?a=commit;h=95098f1b2a08ea7af895b77ce… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Sep 5 23:38:51 2010 +0200 d3d8: An invalid shader version isn't an internal error. --- dlls/d3d8/d3d8_main.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/d3d8/d3d8_main.c b/dlls/d3d8/d3d8_main.c index c471049..6cefe57 100644 --- a/dlls/d3d8/d3d8_main.c +++ b/dlls/d3d8/d3d8_main.c @@ -98,7 +98,7 @@ HRESULT WINAPI ValidateVertexShader(DWORD* vertexshader, DWORD* reserved1, DWORD ret=S_OK; break; default: - ERR("vertexshader version mismatch\n"); + WARN("Invalid shader version token %#x.\n", *vertexshader); ret=E_FAIL; } @@ -136,7 +136,7 @@ HRESULT WINAPI ValidatePixelShader(DWORD* pixelshader, DWORD* reserved1, BOOL bo ret=S_OK; break; default: - ERR("pixelshader version mismatch\n"); + WARN("Invalid shader version token %#x.\n", *pixelshader); ret=E_FAIL; } return ret;

1 0

Henri Verbeet : wined3d: Don't overwrite the offscreen depth/ stencil surface with undefined data in surface_load_ds_location().
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 8fbf23f0ccd1f210204fd3695513f469bad8ada8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8fbf23f0ccd1f210204fd3695… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Sep 5 23:38:50 2010 +0200 wined3d: Don't overwrite the offscreen depth/stencil surface with undefined data in surface_load_ds_location(). --- dlls/wined3d/surface.c | 24 ++++++++++++++---------- 1 files changed, 14 insertions(+), 10 deletions(-) diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index 8f39a97..e60fb44 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -4166,6 +4166,14 @@ void surface_load_ds_location(IWineD3DSurfaceImpl *surface, struct wined3d_conte { GLint old_binding = 0; GLenum bind_target; + GLsizei w, h; + + /* The render target is allowed to be smaller than the depth/stencil + * buffer, so the onscreen depth/stencil buffer is potentially smaller + * than the offscreen surface. Don't overwrite the offscreen surface + * with undefined data. */ + w = min(surface->currentDesc.Width, context->swapchain->presentParms.BackBufferWidth); + h = min(surface->currentDesc.Height, context->swapchain->presentParms.BackBufferHeight); TRACE("Copying onscreen depth buffer to depth texture.\n"); @@ -4190,8 +4198,7 @@ void surface_load_ds_location(IWineD3DSurfaceImpl *surface, struct wined3d_conte bind_target = GL_TEXTURE_2D; } glBindTexture(bind_target, device->depth_blt_texture); - glCopyTexImage2D(bind_target, surface->texture_level, surface->resource.format->glInternal, - 0, 0, surface->currentDesc.Width, surface->currentDesc.Height, 0); + glCopyTexImage2D(bind_target, surface->texture_level, surface->resource.format->glInternal, 0, 0, w, h, 0); glTexParameteri(bind_target, GL_TEXTURE_MIN_FILTER, GL_NEAREST); glTexParameteri(bind_target, GL_TEXTURE_MAG_FILTER, GL_NEAREST); glTexParameteri(bind_target, GL_TEXTURE_WRAP_S, GL_CLAMP_TO_EDGE); @@ -4206,16 +4213,14 @@ void surface_load_ds_location(IWineD3DSurfaceImpl *surface, struct wined3d_conte gl_info->fbo_ops.glGenRenderbuffers(1, &device->depth_blt_rb); checkGLcall("glGenRenderbuffersEXT"); } - if (device->depth_blt_rb_w != surface->currentDesc.Width - || device->depth_blt_rb_h != surface->currentDesc.Height) + if (device->depth_blt_rb_w != w || device->depth_blt_rb_h != h) { gl_info->fbo_ops.glBindRenderbuffer(GL_RENDERBUFFER, device->depth_blt_rb); checkGLcall("glBindRenderbufferEXT"); - gl_info->fbo_ops.glRenderbufferStorage(GL_RENDERBUFFER, GL_RGBA8, - surface->currentDesc.Width, surface->currentDesc.Height); + gl_info->fbo_ops.glRenderbufferStorage(GL_RENDERBUFFER, GL_RGBA8, w, h); checkGLcall("glRenderbufferStorageEXT"); - device->depth_blt_rb_w = surface->currentDesc.Width; - device->depth_blt_rb_h = surface->currentDesc.Height; + device->depth_blt_rb_w = w; + device->depth_blt_rb_h = h; } context_bind_fbo(context, GL_FRAMEBUFFER, &context->dst_fbo); @@ -4225,8 +4230,7 @@ void surface_load_ds_location(IWineD3DSurfaceImpl *surface, struct wined3d_conte context_attach_depth_stencil_fbo(context, GL_FRAMEBUFFER, surface, FALSE); /* Do the actual blit */ - surface_depth_blt(surface, gl_info, device->depth_blt_texture, - surface->currentDesc.Width, surface->currentDesc.Height, bind_target); + surface_depth_blt(surface, gl_info, device->depth_blt_texture, w, h, bind_target); checkGLcall("depth_blt"); if (context->current_fbo) context_bind_fbo(context, GL_FRAMEBUFFER, &context->current_fbo->id);

1 0

Henri Verbeet : wined3d: Translate before scaling for rhw as well in transform_projection().
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 952dfe30be68848fb3799ddb4e732f0e9c7fdbe2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=952dfe30be68848fb3799ddb4… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Sep 5 23:38:49 2010 +0200 wined3d: Translate before scaling for rhw as well in transform_projection(). --- dlls/wined3d/state.c | 12 +++++++----- 1 files changed, 7 insertions(+), 5 deletions(-) diff --git a/dlls/wined3d/state.c b/dlls/wined3d/state.c index 2224247..c04b317 100644 --- a/dlls/wined3d/state.c +++ b/dlls/wined3d/state.c @@ -3905,10 +3905,6 @@ static void transform_projection(DWORD state, IWineD3DStateBlockImpl *stateblock glOrtho(x, x + w, y + h, y, 0.0, -1.0); checkGLcall("glOrtho"); - /* Window Coord 0 is the middle of the first pixel, so translate by 1/2 pixels */ - glTranslatef(63.0f / 128.0f, 63.0f / 128.0f, 0.0f); - checkGLcall("glTranslatef(63.0f / 128.0f, 63.0f / 128.0f, 0.0f)"); - /* D3D texture coordinates are flipped compared to OpenGL ones, so * render everything upside down when rendering offscreen. */ if (context->render_offscreen) @@ -3916,7 +3912,13 @@ static void transform_projection(DWORD state, IWineD3DStateBlockImpl *stateblock glScalef(1.0f, -1.0f, 1.0f); checkGLcall("glScalef"); } - } else { + + /* Window Coord 0 is the middle of the first pixel, so translate by 1/2 pixels */ + glTranslatef(63.0f / 128.0f, 63.0f / 128.0f, 0.0f); + checkGLcall("glTranslatef(63.0f / 128.0f, -63.0f / 128.0f, 0.0f)"); + } + else + { /* The rule is that the window coordinate 0 does not correspond to the beginning of the first pixel, but the center of the first pixel. As a consequence if you want to correctly draw one line exactly from

1 0

Henri Verbeet : wined3d: Do not call device_switch_onscreen_ds() while under the GL lock.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 8e0cb60b092ab3ec5c276d5159e5ff3d8f6f2b23 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8e0cb60b092ab3ec5c276d515… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Sep 5 23:38:48 2010 +0200 wined3d: Do not call device_switch_onscreen_ds() while under the GL lock. --- dlls/wined3d/arb_program_shader.c | 1 + dlls/wined3d/device.c | 9 +++++++++ dlls/wined3d/surface.c | 4 ++++ dlls/wined3d/wined3d_private.h | 3 ++- 4 files changed, 16 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/arb_program_shader.c b/dlls/wined3d/arb_program_shader.c index 7901f0f..3ca5004 100644 --- a/dlls/wined3d/arb_program_shader.c +++ b/dlls/wined3d/arb_program_shader.c @@ -7170,6 +7170,7 @@ HRESULT arbfp_blit_surface(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *src_ return WINED3D_OK; } +/* Do not call while under the GL lock. */ static HRESULT arbfp_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, const WINED3DCOLORVALUE *color) { diff --git a/dlls/wined3d/device.c b/dlls/wined3d/device.c index 5116961..6cd07bb 100644 --- a/dlls/wined3d/device.c +++ b/dlls/wined3d/device.c @@ -576,6 +576,7 @@ void device_get_draw_rect(IWineD3DDeviceImpl *device, RECT *rect) } } +/* Do not call while under the GL lock. */ void device_switch_onscreen_ds(IWineD3DDeviceImpl *device, struct wined3d_context *context, IWineD3DSurfaceImpl *depth_stencil) { @@ -653,6 +654,7 @@ static void prepare_ds_clear(IWineD3DSurfaceImpl *ds, struct wined3d_context *co surface_modify_ds_location(ds, location, ds->ds_current_size.cx, ds->ds_current_size.cy); } +/* Do not call while under the GL lock. */ HRESULT device_clear_render_targets(IWineD3DDeviceImpl *device, UINT rt_count, IWineD3DSurfaceImpl **rts, UINT rect_count, const RECT *rects, const RECT *draw_rect, DWORD flags, const WINED3DCOLORVALUE *color, float depth, DWORD stencil) @@ -715,7 +717,11 @@ HRESULT device_clear_render_targets(IWineD3DDeviceImpl *device, UINT rt_count, I DWORD location = context->render_offscreen ? SFLAG_DS_OFFSCREEN : SFLAG_DS_ONSCREEN; if (location == SFLAG_DS_ONSCREEN && depth_stencil != device->onscreen_depth_stencil) + { + LEAVE_GL(); device_switch_onscreen_ds(device, context, depth_stencil); + ENTER_GL(); + } prepare_ds_clear(depth_stencil, context, location, draw_rect, rect_count, clear_rect); surface_modify_location(depth_stencil, SFLAG_INDRAWABLE, TRUE); @@ -4574,6 +4580,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_Present(IWineD3DDevice *iface, return WINED3D_OK; } +/* Do not call while under the GL lock. */ static HRESULT WINAPI IWineD3DDeviceImpl_Clear(IWineD3DDevice *iface, DWORD rect_count, const RECT *rects, DWORD flags, WINED3DCOLOR color, float depth, DWORD stencil) { @@ -5493,6 +5500,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_DeletePatch(IWineD3DDevice *iface, UINT return WINED3DERR_INVALIDCALL; } +/* Do not call while under the GL lock. */ static HRESULT WINAPI IWineD3DDeviceImpl_ColorFill(IWineD3DDevice *iface, IWineD3DSurface *surface, const RECT *rect, const WINED3DCOLORVALUE *color) { @@ -5511,6 +5519,7 @@ static HRESULT WINAPI IWineD3DDeviceImpl_ColorFill(IWineD3DDevice *iface, return surface_color_fill(s, rect, color); } +/* Do not call while under the GL lock. */ static void WINAPI IWineD3DDeviceImpl_ClearRendertargetView(IWineD3DDevice *iface, IWineD3DRendertargetView *rendertarget_view, const WINED3DCOLORVALUE *color) { diff --git a/dlls/wined3d/surface.c b/dlls/wined3d/surface.c index fe26fd5..8f39a97 100644 --- a/dlls/wined3d/surface.c +++ b/dlls/wined3d/surface.c @@ -3356,6 +3356,7 @@ static void surface_blt_fbo(IWineD3DDeviceImpl *device, const WINED3DTEXTUREFILT context_release(context); } +/* Do not call while under the GL lock. */ HRESULT surface_color_fill(IWineD3DSurfaceImpl *s, const RECT *rect, const WINED3DCOLORVALUE *color) { IWineD3DDeviceImpl *device = s->resource.device; @@ -3373,6 +3374,7 @@ HRESULT surface_color_fill(IWineD3DSurfaceImpl *s, const RECT *rect, const WINED } /* Not called from the VTable */ +/* Do not call while under the GL lock. */ static HRESULT IWineD3DSurfaceImpl_BltOverride(IWineD3DSurfaceImpl *dst_surface, const RECT *DestRect, IWineD3DSurfaceImpl *src_surface, const RECT *SrcRect, DWORD Flags, const WINEDDBLTFX *DDBltFx, WINED3DTEXTUREFILTERTYPE Filter) @@ -4873,6 +4875,7 @@ static BOOL ffp_blit_supported(const struct wined3d_gl_info *gl_info, enum blit_ return FALSE; } +/* Do not call while under the GL lock. */ static HRESULT ffp_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, const WINED3DCOLORVALUE *color) { @@ -4924,6 +4927,7 @@ static BOOL cpu_blit_supported(const struct wined3d_gl_info *gl_info, enum blit_ return FALSE; } +/* Do not call while under the GL lock. */ static HRESULT cpu_blit_color_fill(IWineD3DDeviceImpl *device, IWineD3DSurfaceImpl *dst_surface, const RECT *dst_rect, const WINED3DCOLORVALUE *color) { diff --git a/dlls/wined3d/wined3d_private.h b/dlls/wined3d/wined3d_private.h index 691b8cc..2963933 100644 --- a/dlls/wined3d/wined3d_private.h +++ b/dlls/wined3d/wined3d_private.h @@ -1133,7 +1133,8 @@ enum blit_operation BLIT_OP_COLOR_FILL }; -/* Shaders for color conversions in blits */ +/* Shaders for color conversions in blits. Do not do blit operations while + * already under the GL lock. */ struct blit_shader { HRESULT (*alloc_private)(IWineD3DDevice *iface);

1 0

Henri Verbeet : wined3d: Fix a silly typo.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: 6471004a8c10db36c7e33e8372e5cab0f22a1f16 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6471004a8c10db36c7e33e837… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Sun Sep 5 23:38:47 2010 +0200 wined3d: Fix a silly typo. --- dlls/wined3d/directx.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/wined3d/directx.c b/dlls/wined3d/directx.c index 158bdc2..dcd0b85 100644 --- a/dlls/wined3d/directx.c +++ b/dlls/wined3d/directx.c @@ -2362,7 +2362,7 @@ static BOOL IWineD3DImpl_FillGLCaps(struct wined3d_adapter *adapter) TRACE_(d3d_caps)(" IMPLIED: ARB_vertex_array_bgra support (by EXT_vertex_array_bgra).\n"); gl_info->supported[ARB_VERTEX_ARRAY_BGRA] = TRUE; } - if (!gl_info->supported[ARB_TEXTURE_COMPRESSION_RGTC && gl_info->supported[EXT_TEXTURE_COMPRESSION_RGTC]]) + if (!gl_info->supported[ARB_TEXTURE_COMPRESSION_RGTC] && gl_info->supported[EXT_TEXTURE_COMPRESSION_RGTC]) { TRACE_(d3d_caps)(" IMPLIED: ARB_texture_compression_rgtc support (by EXT_texture_compression_rgtc).\n"); gl_info->supported[ARB_TEXTURE_COMPRESSION_RGTC] = TRUE;

1 0

Louis Lenders : kernel32: Change SetHandleCount to match behaviour on versions >WINNT.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: e793f9b0721d84de5f32518e1006efbcb1486b01 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e793f9b0721d84de5f32518e1… Author: Louis Lenders <xerox_xerox2000(a)yahoo.co.uk> Date: Fri Sep 3 20:47:30 2010 +0200 kernel32: Change SetHandleCount to match behaviour on versions >WINNT. --- dlls/kernel32/file.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/kernel32/file.c b/dlls/kernel32/file.c index dadaea6..e32d75d 100644 --- a/dlls/kernel32/file.c +++ b/dlls/kernel32/file.c @@ -1192,7 +1192,7 @@ BOOL WINAPI UnlockFileEx( HANDLE hFile, DWORD reserved, DWORD count_low, DWORD c */ UINT WINAPI SetHandleCount( UINT count ) { - return min( 256, count ); + return count; }

1 0

Octavian Voicu : include: Fix typo in propkeydef.h.
by Alexandre Julliard 06 Sep '10

06 Sep '10

Module: wine Branch: master Commit: a161d5950421d869ec404e61c4c4855079661cfc URL: http://source.winehq.org/git/wine.git/?a=commit;h=a161d5950421d869ec404e61c… Author: Octavian Voicu <octavian.voicu(a)gmail.com> Date: Fri Sep 3 20:44:28 2010 +0300 include: Fix typo in propkeydef.h. --- include/propkeydef.h | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/include/propkeydef.h b/include/propkeydef.h index b3a2559..2a796ea 100644 --- a/include/propkeydef.h +++ b/include/propkeydef.h @@ -60,7 +60,7 @@ #ifdef __cplusplus inline bool operator==(REFPROPERTYKEY guidOne, REFPROPERTYKEY guidOther) { - return IsEqualPropertyKey(guidOne, guidTwo); + return IsEqualPropertyKey(guidOne, guidOther); } inline bool operator!=(REFPROPERTYKEY guidOne, REFPROPERTYKEY guidOther) {

1 0

← Newer
1
...
71
72
73
74
75
76
77
...
86
Older →

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

2003

2002

2001

wine-commits September 2010