July 2012 - Wine-Commits - list.winehq.org

Dmitry Timoshkov : gdiplus: Add a test to show that GdipCloneImage is not supposed to increase refcount of the source image .
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: c7ea378f6a26e50766de52d80f01fab9a77534dc URL: http://source.winehq.org/git/wine.git/?a=commit;h=c7ea378f6a26e50766de52d80… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jul 2 12:57:30 2012 +0900 gdiplus: Add a test to show that GdipCloneImage is not supposed to increase refcount of the source image. --- dlls/gdiplus/tests/image.c | 20 ++++++++++++++++++-- 1 files changed, 18 insertions(+), 2 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index 8c972ee..b6cc322 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -2674,9 +2674,9 @@ static GpImage *load_image(const BYTE *image_data, UINT image_size) BYTE *data; HRESULT hr; GpStatus status; - GpImage *image = NULL; + GpImage *image = NULL, *clone; ImageType image_type; - LONG refcount; + LONG refcount, old_refcount; hmem = GlobalAlloc(0, image_size); data = GlobalLock(hmem); @@ -2696,6 +2696,22 @@ static GpImage *load_image(const BYTE *image_data, UINT image_size) status = GdipGetImageType(image, &image_type); ok(status == Ok, "GdipGetImageType error %d\n", status); + refcount = obj_refcount(stream); + if (image_type == ImageTypeBitmap) + ok(refcount > 1, "expected stream refcount > 1, got %d\n", refcount); + else + ok(refcount == 1, "expected stream refcount 1, got %d\n", refcount); + old_refcount = refcount; + + status = GdipCloneImage(image, &clone); + ok(status == Ok, "GdipCloneImage error %d\n", status); + refcount = obj_refcount(stream); + ok(refcount == old_refcount, "expected stream refcount %d, got %d\n", old_refcount, refcount); + status = GdipDisposeImage(clone); + ok(status == Ok, "GdipDisposeImage error %d\n", status); + refcount = obj_refcount(stream); + ok(refcount == old_refcount, "expected stream refcount %d, got %d\n", old_refcount, refcount); + refcount = IStream_Release(stream); if (image_type == ImageTypeBitmap) ok(refcount >= 1, "expected stream refcount != 0\n");

1 0

Dmitry Timoshkov : gdiplus: Add some tests for GdipGetPropertySize and GdipGetAllPropertyItems.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: b61a1ba3226e41b82dc8960464505b284fd1dd84 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b61a1ba3226e41b82dc896046… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Jul 2 13:00:20 2012 +0900 gdiplus: Add some tests for GdipGetPropertySize and GdipGetAllPropertyItems. --- dlls/gdiplus/tests/image.c | 169 ++++++++++++++++++++++++++++++++++++++++++++ 1 files changed, 169 insertions(+), 0 deletions(-) diff --git a/dlls/gdiplus/tests/image.c b/dlls/gdiplus/tests/image.c index c49276d..c001a86 100644 --- a/dlls/gdiplus/tests/image.c +++ b/dlls/gdiplus/tests/image.c @@ -3127,6 +3127,174 @@ static void test_tiff_properties(void) GdipDisposeImage(image); } +static void test_GdipGetAllPropertyItems(void) +{ + static const struct test_data + { + ULONG type, id, length; + BYTE value[32]; + } td[16] = + { + { PropertyTagTypeLong, 0xfe, 4, { 0 } }, + { PropertyTagTypeShort, 0x100, 2, { 1 } }, + { PropertyTagTypeShort, 0x101, 2, { 1 } }, + { PropertyTagTypeShort, 0x102, 6, { 8,0,8,0,8,0 } }, + { PropertyTagTypeShort, 0x103, 2, { 1 } }, + { PropertyTagTypeShort, 0x106, 2, { 2,0 } }, + { PropertyTagTypeASCII, 0x10d, 27, "/home/meh/Desktop/test.tif" }, + { PropertyTagTypeLong, 0x111, 4, { 8,0,0,0 } }, + { PropertyTagTypeShort, 0x112, 2, { 1 } }, + { PropertyTagTypeShort, 0x115, 2, { 3,0 } }, + { PropertyTagTypeShort, 0x116, 2, { 0x40,0 } }, + { PropertyTagTypeLong, 0x117, 4, { 3,0,0,0 } }, + { PropertyTagTypeRational, 0x11a, 8, { 0,0,0,72,0,0,0,1 } }, + { PropertyTagTypeRational, 0x11b, 8, { 0,0,0,72,0,0,0,1 } }, + { PropertyTagTypeShort, 0x11c, 2, { 1 } }, + { PropertyTagTypeShort, 0x128, 2, { 2 } } + }; + GpStatus status; + GpImage *image; + GUID guid; + UINT dim_count, frame_count, prop_count, prop_size, i; + UINT total_size, total_count; + PROPID *prop_id; + PropertyItem *prop_item; + const char *item_data; + + image = load_image(tiffimage, sizeof(tiffimage)); + ok(image != 0, "Failed to load TIFF image data\n"); + if (!image) return; + + dim_count = 0xdeadbeef; + status = GdipImageGetFrameDimensionsCount(image, &dim_count); + expect(Ok, status); + expect(1, dim_count); + + status = GdipImageGetFrameDimensionsList(image, &guid, 1); + expect(Ok, status); + expect_guid(&FrameDimensionPage, &guid, __LINE__, FALSE); + + frame_count = 0xdeadbeef; + status = GdipImageGetFrameCount(image, &guid, &frame_count); + expect(Ok, status); + expect(1, frame_count); + + prop_count = 0xdeadbeef; + status = GdipGetPropertyCount(image, &prop_count); + expect(Ok, status); + ok(prop_count == sizeof(td)/sizeof(td[0]), + "expected property count %u, got %u\n", (UINT)(sizeof(td)/sizeof(td[0])), prop_count); + + prop_id = HeapAlloc(GetProcessHeap(), 0, prop_count * sizeof(*prop_id)); + + status = GdipGetPropertyIdList(image, prop_count, prop_id); + expect(Ok, status); + + prop_size = 0; + for (i = 0; i < prop_count; i++) + { + UINT size; + status = GdipGetPropertyItemSize(image, prop_id[i], &size); + expect(Ok, status); + if (status != Ok) break; + ok(size > sizeof(*prop_item), "%u: too small item length %u\n", i, size); + + prop_size += size; + + prop_item = HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, size); + status = GdipGetPropertyItem(image, prop_id[i], size, prop_item); + expect(Ok, status); + ok(prop_item->value == prop_item + 1, "expected item->value %p, got %p\n", prop_item + 1, prop_item->value); + ok(td[i].type == prop_item->type, + "%u: expected type %u, got %u\n", i, td[i].type, prop_item->type); + ok(td[i].id == prop_item->id, "%u: expected id %#x, got %#x\n", i, td[i].id, prop_item->id); + size -= sizeof(*prop_item); + ok(prop_item->length == size, "%u: expected length %u, got %u\n", i, size, prop_item->length); + ok(td[i].length == prop_item->length, "%u: expected length %u, got %u\n", i, td[i].length, prop_item->length); + if (td[i].length == prop_item->length) + { + int match = memcmp(td[i].value, prop_item->value, td[i].length) == 0; + ok(match, "%u: data mismatch\n", i); + if (!match) + { + UINT j; + BYTE *data = prop_item->value; + printf("id %#x:", prop_item->id); + for (j = 0; j < prop_item->length; j++) + printf(" %02x", data[j]); + printf("\n"); + } + } + HeapFree(GetProcessHeap(), 0, prop_item); + } + + HeapFree(GetProcessHeap(), 0, prop_id); + + status = GdipGetPropertySize(NULL, &total_size, &total_count); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, &total_size, NULL); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, NULL, &total_count); + expect(InvalidParameter, status); + status = GdipGetPropertySize(image, NULL, NULL); + expect(InvalidParameter, status); + total_size = 0xdeadbeef; + total_count = 0xdeadbeef; + status = GdipGetPropertySize(image, &total_size, &total_count); + expect(Ok, status); + ok(prop_count == total_count, + "expected total property count %u, got %u\n", prop_count, total_count); + ok(prop_size == total_size, + "expected total property size %u, got %u\n", prop_size, total_size); + + prop_item = HeapAlloc(GetProcessHeap(), 0, prop_size); + + status = GdipGetAllPropertyItems(image, 0, prop_count, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, 1, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, 0, 0, NULL); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size + 1, prop_count, prop_item); + expect(InvalidParameter, status); + status = GdipGetAllPropertyItems(image, prop_size, prop_count, prop_item); + expect(Ok, status); + + item_data = (const char *)(prop_item + prop_count); + for (i = 0; i < prop_count; i++) + { + ok(prop_item[i].value == item_data, "%u: expected value %p, got %p\n", + i, item_data, prop_item[i].value); + ok(td[i].type == prop_item[i].type, + "%u: expected type %u, got %u\n", i, td[i].type, prop_item[i].type); + ok(td[i].id == prop_item[i].id, "%u: expected id %#x, got %#x\n", i, td[i].id, prop_item[i].id); + ok(td[i].length == prop_item[i].length, "%u: expected length %u, got %u\n", i, td[i].length, prop_item[i].length); + if (td[i].length == prop_item[i].length) + { + int match = memcmp(td[i].value, prop_item[i].value, td[i].length) == 0; + ok(match, "%u: data mismatch\n", i); + if (!match) + { + UINT j; + BYTE *data = prop_item[i].value; + printf("id %#x:", prop_item[i].id); + for (j = 0; j < prop_item[i].length; j++) + printf(" %02x", data[j]); + printf("\n"); + } + } + item_data += prop_item[i].length; + } + + HeapFree(GetProcessHeap(), 0, prop_item); + + GdipDisposeImage(image); +} + START_TEST(image) { struct GdiplusStartupInput gdiplusStartupInput; @@ -3139,6 +3307,7 @@ START_TEST(image) GdiplusStartup(&gdiplusToken, &gdiplusStartupInput, NULL); + test_GdipGetAllPropertyItems(); test_tiff_properties(); test_image_properties(); test_Scan0();

1 0

Michael Stefaniuc : quartz/tests: Add COM aggregation test for NullRenderer .
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 8309a38d6aeb5b885d7905ab58cc9149725ca8ef URL: http://source.winehq.org/git/wine.git/?a=commit;h=8309a38d6aeb5b885d7905ab5… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:25:43 2012 +0200 quartz/tests: Add COM aggregation test for NullRenderer. --- dlls/quartz/tests/misc.c | 19 +++++++++++++++++++ 1 files changed, 19 insertions(+), 0 deletions(-) diff --git a/dlls/quartz/tests/misc.c b/dlls/quartz/tests/misc.c index 74c386b..748c0f2 100644 --- a/dlls/quartz/tests/misc.c +++ b/dlls/quartz/tests/misc.c @@ -69,6 +69,11 @@ static void test_aggregation(const CLSID clsidOuter, const CLSID clsidInner, /* for aggregation, we should only be able to request IUnknown */ hr = CoCreateInstance(&clsidInner, pUnkOuter, CLSCTX_INPROC_SERVER, &iidInner, (LPVOID*)&pUnkInnerFail); + if (hr == REGDB_E_CLASSNOTREG) + { + skip("Class not registered\n"); + return; + } ok(hr == E_NOINTERFACE, "CoCreateInstance returned %x\n", hr); ok(pUnkInnerFail == NULL, "pUnkInnerFail is not NULL\n"); @@ -157,6 +162,19 @@ static void test_aggregation(const CLSID clsidOuter, const CLSID clsidInner, } while (refCount); } +static void test_null_renderer_aggregations(void) +{ + const IID * iids[] = { + &IID_IMediaFilter, &IID_IBaseFilter + }; + int i; + + for (i = 0; i < sizeof(iids) / sizeof(iids[0]); i++) + { + test_aggregation(CLSID_SystemClock, CLSID_NullRenderer, IID_IReferenceClock, *iids[i]); + } +} + static void test_video_renderer_aggregations(void) { const IID * iids[] = { @@ -206,6 +224,7 @@ START_TEST(misc) { CoInitialize(NULL); + test_null_renderer_aggregations(); test_video_renderer_aggregations(); test_filter_graph_aggregations(); test_filter_mapper_aggregations();

1 0

Michael Stefaniuc : quartz: Move VideoRenderer_create() to avoid forward declarations.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 02216db8ca82c5e370dcc788c70c652bbcd6add3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=02216db8ca82c5e370dcc788c… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:07:46 2012 +0200 quartz: Move VideoRenderer_create() to avoid forward declarations. --- dlls/quartz/videorenderer.c | 124 +++++++++++++++++++++--------------------- 1 files changed, 62 insertions(+), 62 deletions(-) diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c index 69437a1..a045ba8 100644 --- a/dlls/quartz/videorenderer.c +++ b/dlls/quartz/videorenderer.c @@ -42,12 +42,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(quartz); -static const IBaseFilterVtbl VideoRenderer_Vtbl; -static const IUnknownVtbl IInner_VTable; -static const IBasicVideoVtbl IBasicVideo_VTable; -static const IVideoWindowVtbl IVideoWindow_VTable; -static const IAMFilterMiscFlagsVtbl IAMFilterMiscFlags_Vtbl; - typedef struct VideoRendererImpl { BaseRenderer renderer; @@ -640,62 +634,6 @@ static const BaseControlVideoFuncTable renderer_BaseControlVideoFuncTable = { VideoRenderer_SetTargetRect }; -HRESULT VideoRenderer_create(IUnknown * pUnkOuter, LPVOID * ppv) -{ - HRESULT hr; - VideoRendererImpl * pVideoRenderer; - - TRACE("(%p, %p)\n", pUnkOuter, ppv); - - *ppv = NULL; - - pVideoRenderer = CoTaskMemAlloc(sizeof(VideoRendererImpl)); - pVideoRenderer->IUnknown_inner.lpVtbl = &IInner_VTable; - pVideoRenderer->IAMFilterMiscFlags_iface.lpVtbl = &IAMFilterMiscFlags_Vtbl; - - pVideoRenderer->init = 0; - ZeroMemory(&pVideoRenderer->SourceRect, sizeof(RECT)); - ZeroMemory(&pVideoRenderer->DestRect, sizeof(RECT)); - ZeroMemory(&pVideoRenderer->WindowPos, sizeof(RECT)); - - if (pUnkOuter) - pVideoRenderer->outer_unk = pUnkOuter; - else - pVideoRenderer->outer_unk = &pVideoRenderer->IUnknown_inner; - - hr = BaseRenderer_Init(&pVideoRenderer->renderer, &VideoRenderer_Vtbl, pUnkOuter, &CLSID_VideoRenderer, (DWORD_PTR)(__FILE__ ": VideoRendererImpl.csFilter"), &BaseFuncTable); - - if (FAILED(hr)) - goto fail; - - hr = BaseControlWindow_Init(&pVideoRenderer->baseControlWindow, &IVideoWindow_VTable, &pVideoRenderer->renderer.filter, &pVideoRenderer->renderer.filter.csFilter, &pVideoRenderer->renderer.pInputPin->pin, &renderer_BaseWindowFuncTable); - if (FAILED(hr)) - goto fail; - - hr = BaseControlVideo_Init(&pVideoRenderer->baseControlVideo, &IBasicVideo_VTable, &pVideoRenderer->renderer.filter, &pVideoRenderer->renderer.filter.csFilter, &pVideoRenderer->renderer.pInputPin->pin, &renderer_BaseControlVideoFuncTable); - if (FAILED(hr)) - goto fail; - - if (!CreateRenderingSubsystem(pVideoRenderer)) { - hr = E_FAIL; - goto fail; - } - - *ppv = &pVideoRenderer->IUnknown_inner; - return S_OK; - -fail: - BaseRendererImpl_Release(&pVideoRenderer->renderer.filter.IBaseFilter_iface); - CoTaskMemFree(pVideoRenderer); - return hr; -} - -HRESULT VideoRendererDefault_create(IUnknown * pUnkOuter, LPVOID * ppv) -{ - /* TODO: Attempt to use the VMR-7 renderer instead when possible */ - return VideoRenderer_create(pUnkOuter, ppv); -} - static inline VideoRendererImpl *impl_from_IUnknown(IUnknown *iface) { return CONTAINING_RECORD(iface, VideoRendererImpl, IUnknown_inner); @@ -1060,3 +998,65 @@ static const IAMFilterMiscFlagsVtbl IAMFilterMiscFlags_Vtbl = { AMFilterMiscFlags_Release, AMFilterMiscFlags_GetMiscFlags }; + +HRESULT VideoRenderer_create(IUnknown *pUnkOuter, void **ppv) +{ + HRESULT hr; + VideoRendererImpl * pVideoRenderer; + + TRACE("(%p, %p)\n", pUnkOuter, ppv); + + *ppv = NULL; + + pVideoRenderer = CoTaskMemAlloc(sizeof(VideoRendererImpl)); + pVideoRenderer->IUnknown_inner.lpVtbl = &IInner_VTable; + pVideoRenderer->IAMFilterMiscFlags_iface.lpVtbl = &IAMFilterMiscFlags_Vtbl; + + pVideoRenderer->init = 0; + ZeroMemory(&pVideoRenderer->SourceRect, sizeof(RECT)); + ZeroMemory(&pVideoRenderer->DestRect, sizeof(RECT)); + ZeroMemory(&pVideoRenderer->WindowPos, sizeof(RECT)); + + if (pUnkOuter) + pVideoRenderer->outer_unk = pUnkOuter; + else + pVideoRenderer->outer_unk = &pVideoRenderer->IUnknown_inner; + + hr = BaseRenderer_Init(&pVideoRenderer->renderer, &VideoRenderer_Vtbl, pUnkOuter, + &CLSID_VideoRenderer, (DWORD_PTR)(__FILE__ ": VideoRendererImpl.csFilter"), + &BaseFuncTable); + + if (FAILED(hr)) + goto fail; + + hr = BaseControlWindow_Init(&pVideoRenderer->baseControlWindow, &IVideoWindow_VTable, + &pVideoRenderer->renderer.filter, &pVideoRenderer->renderer.filter.csFilter, + &pVideoRenderer->renderer.pInputPin->pin, &renderer_BaseWindowFuncTable); + if (FAILED(hr)) + goto fail; + + hr = BaseControlVideo_Init(&pVideoRenderer->baseControlVideo, &IBasicVideo_VTable, + &pVideoRenderer->renderer.filter, &pVideoRenderer->renderer.filter.csFilter, + &pVideoRenderer->renderer.pInputPin->pin, &renderer_BaseControlVideoFuncTable); + if (FAILED(hr)) + goto fail; + + if (!CreateRenderingSubsystem(pVideoRenderer)) { + hr = E_FAIL; + goto fail; + } + + *ppv = &pVideoRenderer->IUnknown_inner; + return S_OK; + +fail: + BaseRendererImpl_Release(&pVideoRenderer->renderer.filter.IBaseFilter_iface); + CoTaskMemFree(pVideoRenderer); + return hr; +} + +HRESULT VideoRendererDefault_create(IUnknown * pUnkOuter, LPVOID * ppv) +{ + /* TODO: Attempt to use the VMR-7 renderer instead when possible */ + return VideoRenderer_create(pUnkOuter, ppv); +}

1 0

Michael Stefaniuc : quartz: Standardize COM aggregation for VideoRenderer.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 37cfaa2d026ec71ed8d522bfbe7f93c70199367b URL: http://source.winehq.org/git/wine.git/?a=commit;h=37cfaa2d026ec71ed8d522bfb… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:06:01 2012 +0200 quartz: Standardize COM aggregation for VideoRenderer. --- dlls/quartz/quartz_private.h | 2 - dlls/quartz/videorenderer.c | 101 +++++++++++++++-------------------------- 2 files changed, 37 insertions(+), 66 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=37cfaa2d026ec71ed8d52…

1 0

Michael Stefaniuc : quartz: Move NullRenderer_create() to avoid forward declarations.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 67cb8da5cb8f8ee0d9cf770d567f20f4e0238ab5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=67cb8da5cb8f8ee0d9cf770d5… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:03:21 2012 +0200 quartz: Move NullRenderer_create() to avoid forward declarations. --- dlls/quartz/nullrenderer.c | 70 ++++++++++++++++++++----------------------- 1 files changed, 33 insertions(+), 37 deletions(-) diff --git a/dlls/quartz/nullrenderer.c b/dlls/quartz/nullrenderer.c index 6d93dca..16a24ac 100644 --- a/dlls/quartz/nullrenderer.c +++ b/dlls/quartz/nullrenderer.c @@ -41,10 +41,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(quartz); -static const IBaseFilterVtbl NullRenderer_Vtbl; -static const IUnknownVtbl IInner_VTable; -static const IAMFilterMiscFlagsVtbl IAMFilterMiscFlags_Vtbl; - typedef struct NullRendererImpl { BaseRenderer renderer; @@ -86,39 +82,6 @@ static const BaseRendererFuncTable RendererFuncTable = { NULL, }; -HRESULT NullRenderer_create(IUnknown * pUnkOuter, LPVOID * ppv) -{ - HRESULT hr; - NullRendererImpl * pNullRenderer; - - TRACE("(%p, %p)\n", pUnkOuter, ppv); - - *ppv = NULL; - - pNullRenderer = CoTaskMemAlloc(sizeof(NullRendererImpl)); - pNullRenderer->IUnknown_inner.lpVtbl = &IInner_VTable; - pNullRenderer->IAMFilterMiscFlags_iface.lpVtbl = &IAMFilterMiscFlags_Vtbl; - - if (pUnkOuter) - pNullRenderer->outer_unk = pUnkOuter; - else - pNullRenderer->outer_unk = &pNullRenderer->IUnknown_inner; - - hr = BaseRenderer_Init(&pNullRenderer->renderer, &NullRenderer_Vtbl, pUnkOuter, - &CLSID_NullRenderer, (DWORD_PTR)(__FILE__ ": NullRendererImpl.csFilter"), - &RendererFuncTable); - - if (FAILED(hr)) - { - BaseRendererImpl_Release(&pNullRenderer->renderer.filter.IBaseFilter_iface); - CoTaskMemFree(pNullRenderer); - } - else - *ppv = &pNullRenderer->IUnknown_inner; - - return S_OK; -} - static inline NullRendererImpl *impl_from_IUnknown(IUnknown *iface) { return CONTAINING_RECORD(iface, NullRendererImpl, IUnknown_inner); @@ -260,3 +223,36 @@ static const IAMFilterMiscFlagsVtbl IAMFilterMiscFlags_Vtbl = { AMFilterMiscFlags_Release, AMFilterMiscFlags_GetMiscFlags }; + +HRESULT NullRenderer_create(IUnknown *pUnkOuter, void **ppv) +{ + HRESULT hr; + NullRendererImpl *pNullRenderer; + + TRACE("(%p, %p)\n", pUnkOuter, ppv); + + *ppv = NULL; + + pNullRenderer = CoTaskMemAlloc(sizeof(NullRendererImpl)); + pNullRenderer->IUnknown_inner.lpVtbl = &IInner_VTable; + pNullRenderer->IAMFilterMiscFlags_iface.lpVtbl = &IAMFilterMiscFlags_Vtbl; + + if (pUnkOuter) + pNullRenderer->outer_unk = pUnkOuter; + else + pNullRenderer->outer_unk = &pNullRenderer->IUnknown_inner; + + hr = BaseRenderer_Init(&pNullRenderer->renderer, &NullRenderer_Vtbl, pUnkOuter, + &CLSID_NullRenderer, (DWORD_PTR)(__FILE__ ": NullRendererImpl.csFilter"), + &RendererFuncTable); + + if (FAILED(hr)) + { + BaseRendererImpl_Release(&pNullRenderer->renderer.filter.IBaseFilter_iface); + CoTaskMemFree(pNullRenderer); + } + else + *ppv = &pNullRenderer->IUnknown_inner; + + return S_OK; +}

1 0

Michael Stefaniuc : quartz: Move the FilterMapper_create to avoid forward declarations.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 7fa17be7c5c8ea8b2f8ff0d7995e050d70209c3f URL: http://source.winehq.org/git/wine.git/?a=commit;h=7fa17be7c5c8ea8b2f8ff0d79… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 00:58:29 2012 +0200 quartz: Move the FilterMapper_create to avoid forward declarations. --- dlls/quartz/filtermapper.c | 93 +++++++++++++++++++++----------------------- 1 files changed, 44 insertions(+), 49 deletions(-) diff --git a/dlls/quartz/filtermapper.c b/dlls/quartz/filtermapper.c index 464caee..51b5ed7 100644 --- a/dlls/quartz/filtermapper.c +++ b/dlls/quartz/filtermapper.c @@ -55,11 +55,6 @@ typedef struct FilterMapper3Impl LONG ref; } FilterMapper3Impl; -static const IUnknownVtbl IInner_VTable; -static const IFilterMapper3Vtbl fm3vtbl; -static const IFilterMapperVtbl fmvtbl; -static const IAMFilterDataVtbl AMFilterDataVtbl; - static inline FilterMapper3Impl *impl_from_IFilterMapper3( IFilterMapper3 *iface ) { return CONTAINING_RECORD(iface, FilterMapper3Impl, IFilterMapper3_iface); @@ -180,50 +175,6 @@ static void delete_vector(struct Vector * v) v->capacity = 0; } -HRESULT FilterMapper2_create(IUnknown *pUnkOuter, LPVOID *ppObj) -{ - FilterMapper3Impl * pFM2impl; - - TRACE("(%p, %p)\n", pUnkOuter, ppObj); - - pFM2impl = CoTaskMemAlloc(sizeof(*pFM2impl)); - if (!pFM2impl) - return E_OUTOFMEMORY; - - pFM2impl->IUnknown_inner.lpVtbl = &IInner_VTable; - pFM2impl->IFilterMapper3_iface.lpVtbl = &fm3vtbl; - pFM2impl->IFilterMapper_iface.lpVtbl = &fmvtbl; - pFM2impl->IAMFilterData_iface.lpVtbl = &AMFilterDataVtbl; - pFM2impl->ref = 1; - - if (pUnkOuter) - pFM2impl->outer_unk = pUnkOuter; - else - pFM2impl->outer_unk = &pFM2impl->IUnknown_inner; - - *ppObj = &pFM2impl->IUnknown_inner; - - TRACE("-- created at %p\n", pFM2impl); - - return S_OK; -} - -HRESULT FilterMapper_create(IUnknown *pUnkOuter, LPVOID *ppObj) -{ - FilterMapper3Impl *pFM2impl; - HRESULT hr; - - TRACE("(%p, %p)\n", pUnkOuter, ppObj); - - hr = FilterMapper2_create(pUnkOuter, (LPVOID*)&pFM2impl); - if (FAILED(hr)) - return hr; - - *ppObj = &pFM2impl->IFilterMapper_iface; - - return hr; -} - /*** IUnknown (inner) methods ***/ static HRESULT WINAPI Inner_QueryInterface(IUnknown *iface, REFIID riid, void **ppv) @@ -1737,3 +1688,47 @@ static const IAMFilterDataVtbl AMFilterDataVtbl = { AMFilterData_ParseFilterData, AMFilterData_CreateFilterData }; + +HRESULT FilterMapper2_create(IUnknown *pUnkOuter, LPVOID *ppObj) +{ + FilterMapper3Impl * pFM2impl; + + TRACE("(%p, %p)\n", pUnkOuter, ppObj); + + pFM2impl = CoTaskMemAlloc(sizeof(*pFM2impl)); + if (!pFM2impl) + return E_OUTOFMEMORY; + + pFM2impl->IUnknown_inner.lpVtbl = &IInner_VTable; + pFM2impl->IFilterMapper3_iface.lpVtbl = &fm3vtbl; + pFM2impl->IFilterMapper_iface.lpVtbl = &fmvtbl; + pFM2impl->IAMFilterData_iface.lpVtbl = &AMFilterDataVtbl; + pFM2impl->ref = 1; + + if (pUnkOuter) + pFM2impl->outer_unk = pUnkOuter; + else + pFM2impl->outer_unk = &pFM2impl->IUnknown_inner; + + *ppObj = &pFM2impl->IUnknown_inner; + + TRACE("-- created at %p\n", pFM2impl); + + return S_OK; +} + +HRESULT FilterMapper_create(IUnknown *pUnkOuter, LPVOID *ppObj) +{ + FilterMapper3Impl *pFM2impl; + HRESULT hr; + + TRACE("(%p, %p)\n", pUnkOuter, ppObj); + + hr = FilterMapper2_create(pUnkOuter, (LPVOID*)&pFM2impl); + if (FAILED(hr)) + return hr; + + *ppObj = &pFM2impl->IFilterMapper_iface; + + return hr; +}

1 0

Michael Stefaniuc : quartz: COM cleanup for IAMFilterMiscFlags in VideoRenderer.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: d28971a23e16b920474cde06b1099a6276092e76 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d28971a23e16b920474cde06b… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:04:04 2012 +0200 quartz: COM cleanup for IAMFilterMiscFlags in VideoRenderer. --- dlls/quartz/videorenderer.c | 30 ++++++++++++++++++------------ 1 files changed, 18 insertions(+), 12 deletions(-) diff --git a/dlls/quartz/videorenderer.c b/dlls/quartz/videorenderer.c index 87f3ea9..3e632cb 100644 --- a/dlls/quartz/videorenderer.c +++ b/dlls/quartz/videorenderer.c @@ -55,7 +55,7 @@ typedef struct VideoRendererImpl BaseControlVideo baseControlVideo; const IUnknownVtbl * IInner_vtbl; - const IAMFilterMiscFlagsVtbl *IAMFilterMiscFlags_vtbl; + IAMFilterMiscFlags IAMFilterMiscFlags_iface; BOOL init; HANDLE hThread; @@ -656,7 +656,7 @@ HRESULT VideoRenderer_create(IUnknown * pUnkOuter, LPVOID * ppv) pVideoRenderer->bUnkOuterValid = FALSE; pVideoRenderer->bAggregatable = FALSE; pVideoRenderer->IInner_vtbl = &IInner_VTable; - pVideoRenderer->IAMFilterMiscFlags_vtbl = &IAMFilterMiscFlags_Vtbl; + pVideoRenderer->IAMFilterMiscFlags_iface.lpVtbl = &IAMFilterMiscFlags_Vtbl; pVideoRenderer->init = 0; ZeroMemory(&pVideoRenderer->SourceRect, sizeof(RECT)); @@ -711,7 +711,7 @@ static HRESULT WINAPI VideoRendererInner_QueryInterface(IUnknown * iface, REFIID else if (IsEqualIID(riid, &IID_IVideoWindow)) *ppv = &This->baseControlWindow.IVideoWindow_iface; else if (IsEqualIID(riid, &IID_IAMFilterMiscFlags)) - *ppv = &This->IAMFilterMiscFlags_vtbl; + *ppv = &This->IAMFilterMiscFlags_iface; else { HRESULT hr; @@ -1050,26 +1050,32 @@ static const IVideoWindowVtbl IVideoWindow_VTable = BaseControlWindowImpl_IsCursorHidden }; -static VideoRendererImpl *from_IAMFilterMiscFlags(IAMFilterMiscFlags *iface) { - return (VideoRendererImpl*)((char*)iface - offsetof(VideoRendererImpl, IAMFilterMiscFlags_vtbl)); +static VideoRendererImpl *impl_from_IAMFilterMiscFlags(IAMFilterMiscFlags *iface) +{ + return CONTAINING_RECORD(iface, VideoRendererImpl, IAMFilterMiscFlags_iface); } -static HRESULT WINAPI AMFilterMiscFlags_QueryInterface(IAMFilterMiscFlags *iface, REFIID riid, void **ppv) { - VideoRendererImpl *This = from_IAMFilterMiscFlags(iface); +static HRESULT WINAPI AMFilterMiscFlags_QueryInterface(IAMFilterMiscFlags *iface, REFIID riid, + void **ppv) +{ + VideoRendererImpl *This = impl_from_IAMFilterMiscFlags(iface); return IUnknown_QueryInterface((IUnknown*)This, riid, ppv); } -static ULONG WINAPI AMFilterMiscFlags_AddRef(IAMFilterMiscFlags *iface) { - VideoRendererImpl *This = from_IAMFilterMiscFlags(iface); +static ULONG WINAPI AMFilterMiscFlags_AddRef(IAMFilterMiscFlags *iface) +{ + VideoRendererImpl *This = impl_from_IAMFilterMiscFlags(iface); return IUnknown_AddRef((IUnknown*)This); } -static ULONG WINAPI AMFilterMiscFlags_Release(IAMFilterMiscFlags *iface) { - VideoRendererImpl *This = from_IAMFilterMiscFlags(iface); +static ULONG WINAPI AMFilterMiscFlags_Release(IAMFilterMiscFlags *iface) +{ + VideoRendererImpl *This = impl_from_IAMFilterMiscFlags(iface); return IUnknown_Release((IUnknown*)This); } -static ULONG WINAPI AMFilterMiscFlags_GetMiscFlags(IAMFilterMiscFlags *iface) { +static ULONG WINAPI AMFilterMiscFlags_GetMiscFlags(IAMFilterMiscFlags *iface) +{ return AM_FILTER_MISC_FLAGS_IS_RENDERER; }

1 0

Alexandre Julliard : gdi32: Fix bitmap glyph allocation size in ExtTextOut fallback implementation.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 4c3cc50104a193dcffa988116b52b9ff3042a5f8 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4c3cc50104a193dcffa988116… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Mon Jul 2 10:58:23 2012 +0200 gdi32: Fix bitmap glyph allocation size in ExtTextOut fallback implementation. --- dlls/gdi32/font.c | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/gdi32/font.c b/dlls/gdi32/font.c index bb97902..5fefa2f 100644 --- a/dlls/gdi32/font.c +++ b/dlls/gdi32/font.c @@ -1799,7 +1799,7 @@ static void draw_glyph( HDC hdc, INT origin_x, INT origin_y, const GLYPHMETRICS const struct gdi_image_bits *image, const RECT *clip ) { static const BYTE masks[8] = {0x80, 0x40, 0x20, 0x10, 0x08, 0x04, 0x02, 0x01}; - UINT x, y, i, count; + UINT x, y, i, count, max_count; BYTE *ptr = image->ptr; int stride = get_dib_stride( metrics->gmBlackBoxX, 1 ); POINT *pts; @@ -1812,8 +1812,8 @@ static void draw_glyph( HDC hdc, INT origin_x, INT origin_y, const GLYPHMETRICS if (!clip) clipped_rect = rect; else if (!intersect_rect( &clipped_rect, &rect, clip )) return; - pts = HeapAlloc( GetProcessHeap(), 0, - max(2,metrics->gmBlackBoxX) * metrics->gmBlackBoxY * sizeof(*pts) ); + max_count = (metrics->gmBlackBoxX + 1) * metrics->gmBlackBoxY; + pts = HeapAlloc( GetProcessHeap(), 0, max_count * sizeof(*pts) ); if (!pts) return; count = 0; @@ -1833,6 +1833,7 @@ static void draw_glyph( HDC hdc, INT origin_x, INT origin_y, const GLYPHMETRICS } } } + assert( count <= max_count ); DPtoLP( hdc, pts, count ); for (i = 0; i < count; i += 2) Polyline( hdc, pts + i, 2 ); HeapFree( GetProcessHeap(), 0, pts );

1 0

Michael Stefaniuc : quartz: Use impl_from_IBaseFilter() instead of casts.
by Alexandre Julliard July 2, 2012

July 2, 2012

Module: wine Branch: master Commit: 4e1302c2fbafe664d9cbe93696e6c04af40d703f URL: http://source.winehq.org/git/wine.git/?a=commit;h=4e1302c2fbafe664d9cbe9369… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Jul 2 01:02:24 2012 +0200 quartz: Use impl_from_IBaseFilter() instead of casts. --- dlls/quartz/nullrenderer.c | 11 ++++++++--- 1 files changed, 8 insertions(+), 3 deletions(-) diff --git a/dlls/quartz/nullrenderer.c b/dlls/quartz/nullrenderer.c index 922d64e..6d93dca 100644 --- a/dlls/quartz/nullrenderer.c +++ b/dlls/quartz/nullrenderer.c @@ -183,21 +183,26 @@ static const IUnknownVtbl IInner_VTable = NullRendererInner_Release }; +static inline NullRendererImpl *impl_from_IBaseFilter(IBaseFilter *iface) +{ + return CONTAINING_RECORD(iface, NullRendererImpl, renderer.filter.IBaseFilter_iface); +} + static HRESULT WINAPI NullRenderer_QueryInterface(IBaseFilter * iface, REFIID riid, LPVOID * ppv) { - NullRendererImpl *This = (NullRendererImpl *)iface; + NullRendererImpl *This = impl_from_IBaseFilter(iface); return IUnknown_QueryInterface(This->outer_unk, riid, ppv); } static ULONG WINAPI NullRenderer_AddRef(IBaseFilter * iface) { - NullRendererImpl *This = (NullRendererImpl *)iface; + NullRendererImpl *This = impl_from_IBaseFilter(iface); return IUnknown_AddRef(This->outer_unk); } static ULONG WINAPI NullRenderer_Release(IBaseFilter * iface) { - NullRendererImpl *This = (NullRendererImpl *)iface; + NullRendererImpl *This = impl_from_IBaseFilter(iface); return IUnknown_Release(This->outer_unk); }

1 0