January 2010 - Wine-Commits

Maarten Lankhorst : Revert "dsound: Only initialize one guid for capture and renderer."
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: ef0a0d8b86b42962d5b8a114097943168f53c251 URL: http://source.winehq.org/git/wine.git/?a=commit;h=ef0a0d8b86b42962d5b8a1140… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Tue Jan 5 22:20:52 2010 +0100 Revert "dsound: Only initialize one guid for capture and renderer." Reverts b474649e0e9491f938f1daec74cdca01343dd97e. Some games use pointers to the guid rather than copying the guid even after the enumeration function returns. Allocating on the stack breaks those games. --- dlls/dsound/capture.c | 11 ++++-- dlls/dsound/dsound.c | 10 +++-- dlls/dsound/dsound_main.c | 76 +++++++++++++++++++++--------------------- dlls/dsound/dsound_private.h | 5 ++- dlls/dsound/propset.c | 14 +++---- 5 files changed, 60 insertions(+), 56 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=ef0a0d8b86b42962d5b8a…

1 0

Marcus Meissner : user32: EM_REPLACESEL - handle OOM error.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 69bf18567f666d9568d68c5379a08451311e161c URL: http://source.winehq.org/git/wine.git/?a=commit;h=69bf18567f666d9568d68c537… Author: Marcus Meissner <marcus(a)jet.franken.de> Date: Wed Jan 6 16:52:46 2010 +0100 user32: EM_REPLACESEL - handle OOM error. --- dlls/user32/edit.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/user32/edit.c b/dlls/user32/edit.c index 20a853e..5bdef8a 100644 --- a/dlls/user32/edit.c +++ b/dlls/user32/edit.c @@ -4525,8 +4525,8 @@ LRESULT EditWndProc_common( HWND hwnd, UINT msg, WPARAM wParam, LPARAM lParam, B { LPSTR textA = (LPSTR)lParam; INT countW = MultiByteToWideChar(CP_ACP, 0, textA, -1, NULL, 0); - if((textW = HeapAlloc(GetProcessHeap(), 0, countW * sizeof(WCHAR)))) - MultiByteToWideChar(CP_ACP, 0, textA, -1, textW, countW); + if (!(textW = HeapAlloc(GetProcessHeap(), 0, countW * sizeof(WCHAR)))) break; + MultiByteToWideChar(CP_ACP, 0, textA, -1, textW, countW); } EDIT_EM_ReplaceSel(es, (BOOL)wParam, textW, TRUE, TRUE);

1 0

David Adam : dmloader: Fix a possible null dereference.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 5e39af2683978d97781b850f4259b5ba3d2ab2b4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5e39af2683978d97781b850f4… Author: David Adam <david.adam.cnrs(a)gmail.com> Date: Wed Jan 6 04:49:39 2010 +0100 dmloader: Fix a possible null dereference. --- configure | 9 ++++++ configure.ac | 1 + dlls/dmloader/loader.c | 2 + dlls/dmloader/tests/Makefile.in | 13 +++++++++ dlls/dmloader/tests/loader.c | 57 +++++++++++++++++++++++++++++++++++++++ 5 files changed, 82 insertions(+), 0 deletions(-) diff --git a/configure b/configure index 4680051..b81612d 100755 --- a/configure +++ b/configure @@ -14505,6 +14505,14 @@ dlls/dmloader/Makefile: dlls/dmloader/Makefile.in dlls/Makedll.rules" ac_config_files="$ac_config_files dlls/dmloader/Makefile" ALL_MAKEFILES="$ALL_MAKEFILES \\ + dlls/dmloader/tests/Makefile" +test "x$enable_tests" != xno && ALL_TEST_DIRS="$ALL_TEST_DIRS \\ + dmloader/tests" +ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS +dlls/dmloader/tests/Makefile: dlls/dmloader/tests/Makefile.in dlls/Maketest.rules" +ac_config_files="$ac_config_files dlls/dmloader/tests/Makefile" + +ALL_MAKEFILES="$ALL_MAKEFILES \\ dlls/dmscript/Makefile" test "x$enable_dmscript" != xno && ALL_DLL_DIRS="$ALL_DLL_DIRS \\ dmscript" @@ -18839,6 +18847,7 @@ do "dlls/dmcompos/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmcompos/Makefile" ;; "dlls/dmime/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmime/Makefile" ;; "dlls/dmloader/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmloader/Makefile" ;; + "dlls/dmloader/tests/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmloader/tests/Makefile" ;; "dlls/dmscript/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmscript/Makefile" ;; "dlls/dmstyle/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmstyle/Makefile" ;; "dlls/dmsynth/Makefile") CONFIG_FILES="$CONFIG_FILES dlls/dmsynth/Makefile" ;; diff --git a/configure.ac b/configure.ac index 417e245..6cae9ba 100644 --- a/configure.ac +++ b/configure.ac @@ -2224,6 +2224,7 @@ WINE_CONFIG_MAKEFILE([dlls/dmband/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL WINE_CONFIG_MAKEFILE([dlls/dmcompos/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/dmime/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/dmloader/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) +WINE_CONFIG_MAKEFILE([dlls/dmloader/tests/Makefile],[dlls/Maketest.rules],[dlls],[ALL_TEST_DIRS],[enable_tests]) WINE_CONFIG_MAKEFILE([dlls/dmscript/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/dmstyle/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) WINE_CONFIG_MAKEFILE([dlls/dmsynth/Makefile],[dlls/Makedll.rules],[dlls],[ALL_DLL_DIRS]) diff --git a/dlls/dmloader/loader.c b/dlls/dmloader/loader.c index 393b6b9..0e9748d 100644 --- a/dlls/dmloader/loader.c +++ b/dlls/dmloader/loader.c @@ -644,6 +644,8 @@ static HRESULT WINAPI IDirectMusicLoaderImpl_IDirectMusicLoader_ReleaseObject (L ICOM_THIS_MULTI(IDirectMusicLoaderImpl, LoaderVtbl, iface); TRACE("(%p, %p)\n", This, pObject); + if(!pObject) return E_POINTER; + /* get descriptor */ DM_STRUCT_INIT(&Desc); IDirectMusicObject_GetDescriptor (pObject, &Desc); diff --git a/dlls/dmloader/tests/Makefile.in b/dlls/dmloader/tests/Makefile.in new file mode 100644 index 0000000..48a08f4 --- /dev/null +++ b/dlls/dmloader/tests/Makefile.in @@ -0,0 +1,13 @@ +TOPSRCDIR = @top_srcdir@ +TOPOBJDIR = ../../.. +SRCDIR = @srcdir@ +VPATH = @srcdir@ +TESTDLL = dmloader.dll +IMPORTS = ole32 kernel32 + +CTESTS = \ + loader.c + +(a)MAKE_TEST_RULES@ + +(a)DEPENDENCIES@ # everything below this line is overwritten by make depend diff --git a/dlls/dmloader/tests/loader.c b/dlls/dmloader/tests/loader.c new file mode 100644 index 0000000..dc1ff7a --- /dev/null +++ b/dlls/dmloader/tests/loader.c @@ -0,0 +1,57 @@ +/* + * Copyright (C) 2010 David Adam + * + * 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 <assert.h> + +#define COBJMACROS + +#include "initguid.h" +#include "dmusici.h" +#include "wine/test.h" + +static void test_release_object(void) +{ + HRESULT hr; + IDirectMusicLoader8* loader = NULL; + + hr = CoInitialize(NULL); + if ( FAILED(hr) ) + { + skip("CoInitialize failed.\n"); + return; + } + + hr = CoCreateInstance(&CLSID_DirectMusicLoader, NULL, CLSCTX_INPROC, &IID_IDirectMusicLoader8, (void**)&loader); + if ( FAILED(hr) ) + { + skip("CoCreateInstance failed.\n"); + CoUninitialize(); + return; + } + + hr = IDirectMusicLoader_ReleaseObject(loader, NULL); + ok(hr == E_POINTER, "Expected E_POINTER, received %#x\n", hr); + + IDirectMusicLoader_Release(loader); + CoUninitialize(); +} + +START_TEST(loader) +{ + test_release_object(); +}

1 0

Henri Verbeet : avifil32: Properly check the required buffer size in AVIFILE_AddFrame().
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: c2051d33d65514eed5ca5116450fe41210663dd6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c2051d33d65514eed5ca51164… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jan 5 21:29:00 2010 +0100 avifil32: Properly check the required buffer size in AVIFILE_AddFrame(). --- dlls/avifil32/avifile.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index 57f8294..48c3662 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -1384,7 +1384,7 @@ static HRESULT AVIFILE_AddFrame(IAVIStreamImpl *This, DWORD ckid, DWORD size, DW This->sInfo.dwFlags |= AVISTREAMINFO_FORMATCHANGES; This->sInfo.dwFormatChangeCount++; - if (This->idxFmtChanges == NULL || This->sInfo.dwFormatChangeCount < This->nIdxFmtChanges) { + if (This->idxFmtChanges == NULL || This->nIdxFmtChanges < This->sInfo.dwFormatChangeCount) { This->nIdxFmtChanges += 16; if (This->idxFmtChanges == NULL) This->idxFmtChanges =

1 0

Henri Verbeet : avifil32: Always add the format change frame to the buffer in AVIFILE_AddFrame().
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 32c3adbbaabacf8bdc84a01ed68794c5dc5f7981 URL: http://source.winehq.org/git/wine.git/?a=commit;h=32c3adbbaabacf8bdc84a01ed… Author: Henri Verbeet <hverbeet(a)codeweavers.com> Date: Tue Jan 5 21:28:59 2010 +0100 avifil32: Always add the format change frame to the buffer in AVIFILE_AddFrame(). It doesn't make sense to only do this when the buffer is resized. --- dlls/avifil32/avifile.c | 18 +++++++++--------- 1 files changed, 9 insertions(+), 9 deletions(-) diff --git a/dlls/avifil32/avifile.c b/dlls/avifil32/avifile.c index c4f191a..57f8294 100644 --- a/dlls/avifil32/avifile.c +++ b/dlls/avifil32/avifile.c @@ -1362,6 +1362,8 @@ static HRESULT WINAPI IAVIStream_fnSetInfo(IAVIStream *iface, static HRESULT AVIFILE_AddFrame(IAVIStreamImpl *This, DWORD ckid, DWORD size, DWORD offset, DWORD flags) { + UINT n; + /* pre-conditions */ assert(This != NULL); @@ -1383,8 +1385,6 @@ static HRESULT AVIFILE_AddFrame(IAVIStreamImpl *This, DWORD ckid, DWORD size, DW This->sInfo.dwFormatChangeCount++; if (This->idxFmtChanges == NULL || This->sInfo.dwFormatChangeCount < This->nIdxFmtChanges) { - UINT n = This->sInfo.dwFormatChangeCount; - This->nIdxFmtChanges += 16; if (This->idxFmtChanges == NULL) This->idxFmtChanges = @@ -1395,15 +1395,15 @@ static HRESULT AVIFILE_AddFrame(IAVIStreamImpl *This, DWORD ckid, DWORD size, DW This->nIdxFmtChanges * sizeof(AVIINDEXENTRY)); if (This->idxFmtChanges == NULL) return AVIERR_MEMORY; + } - This->idxFmtChanges[n].ckid = This->lLastFrame; - This->idxFmtChanges[n].dwFlags = 0; - This->idxFmtChanges[n].dwChunkOffset = offset; - This->idxFmtChanges[n].dwChunkLength = size; + n = This->sInfo.dwFormatChangeCount; + This->idxFmtChanges[n].ckid = This->lLastFrame; + This->idxFmtChanges[n].dwFlags = 0; + This->idxFmtChanges[n].dwChunkOffset = offset; + This->idxFmtChanges[n].dwChunkLength = size; - return AVIERR_OK; - } - break; + return AVIERR_OK; case cktypeWAVEbytes: if (This->paf->fInfo.dwFlags & AVIFILEINFO_TRUSTCKTYPE) flags |= AVIIF_KEYFRAME;

1 0

Andrew Eikum : msi: Implement recursive INSTALLSTATE updates for msi treeview selector.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: d1a916a614b83ff14af82632aa49840bddcf8e45 URL: http://source.winehq.org/git/wine.git/?a=commit;h=d1a916a614b83ff14af82632a… Author: Andrew Eikum <aeikum(a)codeweavers.com> Date: Tue Jan 5 11:42:09 2010 -0600 msi: Implement recursive INSTALLSTATE updates for msi treeview selector. --- dlls/msi/dialog.c | 55 ++++++++++++++++++++++++++++++++++++++++++++-------- 1 files changed, 46 insertions(+), 9 deletions(-) diff --git a/dlls/msi/dialog.c b/dlls/msi/dialog.c index a22091f..5fd4bbd 100644 --- a/dlls/msi/dialog.c +++ b/dlls/msi/dialog.c @@ -166,6 +166,8 @@ static MSIFEATURE *msi_seltree_get_selected_feature( msi_control *control ); #define WM_MSI_DIALOG_CREATE (WM_USER+0x100) #define WM_MSI_DIALOG_DESTROY (WM_USER+0x101) +#define USER_INSTALLSTATE_ALL 0x1000 + static DWORD uiThreadId; static HWND hMsiHiddenWindow; @@ -1876,7 +1878,7 @@ msi_seltree_popup_menu( HWND hwnd, INT x, INT y ) /* FIXME: load strings from resources */ AppendMenuA( hMenu, MF_ENABLED, INSTALLSTATE_LOCAL, "Install feature locally"); - AppendMenuA( hMenu, MF_GRAYED, 0x1000, "Install entire feature"); + AppendMenuA( hMenu, MF_ENABLED, USER_INSTALLSTATE_ALL, "Install entire feature"); AppendMenuA( hMenu, MF_ENABLED, INSTALLSTATE_ADVERTISED, "Install on demand"); AppendMenuA( hMenu, MF_ENABLED, INSTALLSTATE_ABSENT, "Don't install"); r = TrackPopupMenu( hMenu, TPM_LEFTALIGN | TPM_TOPALIGN | TPM_RETURNCMD, @@ -1899,6 +1901,37 @@ msi_seltree_feature_from_item( HWND hwnd, HTREEITEM hItem ) return (MSIFEATURE*) tvi.lParam; } +static void +msi_seltree_update_feature_installstate( HWND hwnd, HTREEITEM hItem, + MSIPACKAGE *package, MSIFEATURE *feature, INSTALLSTATE state ) +{ + msi_feature_set_state( package, feature, state ); + msi_seltree_sync_item_state( hwnd, feature, hItem ); + ACTION_UpdateComponentStates( package, feature->Feature ); +} + +static void +msi_seltree_update_siblings_and_children_installstate( HWND hwnd, HTREEITEM curr, + MSIPACKAGE *package, INSTALLSTATE state) +{ + /* update all siblings */ + do + { + MSIFEATURE *feature; + HTREEITEM child; + + feature = msi_seltree_feature_from_item( hwnd, curr ); + msi_seltree_update_feature_installstate( hwnd, curr, package, feature, state ); + + /* update this sibling's children */ + child = (HTREEITEM)SendMessageW( hwnd, TVM_GETNEXTITEM, (WPARAM)TVGN_CHILD, (LPARAM)curr ); + if (child) + msi_seltree_update_siblings_and_children_installstate( hwnd, child, + package, state ); + } + while ((curr = (HTREEITEM)SendMessageW( hwnd, TVM_GETNEXTITEM, (WPARAM)TVGN_NEXT, (LPARAM)curr ))); +} + static LRESULT msi_seltree_menu( HWND hwnd, HTREEITEM hItem ) { @@ -1931,19 +1964,23 @@ msi_seltree_menu( HWND hwnd, HTREEITEM hItem ) switch (r) { - case INSTALLSTATE_LOCAL: + case USER_INSTALLSTATE_ALL: + r = INSTALLSTATE_LOCAL; + /* fall-through */ case INSTALLSTATE_ADVERTISED: case INSTALLSTATE_ABSENT: - msi_feature_set_state(package, feature, r); + { + HTREEITEM child; + child = (HTREEITEM)SendMessageW( hwnd, TVM_GETNEXTITEM, (WPARAM)TVGN_CHILD, (LPARAM)hItem ); + if (child) + msi_seltree_update_siblings_and_children_installstate( hwnd, child, package, r ); + } + /* fall-through */ + case INSTALLSTATE_LOCAL: + msi_seltree_update_feature_installstate( hwnd, hItem, package, feature, r ); break; - default: - FIXME("select feature and all children\n"); } - /* update */ - msi_seltree_sync_item_state( hwnd, feature, hItem ); - ACTION_UpdateComponentStates( package, feature->Feature ); - return 0; }

1 0

André Hentschel : winedbg: Update link.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 76c518539f3a62516770d3eafcc5b112f8ee9e0a URL: http://source.winehq.org/git/wine.git/?a=commit;h=76c518539f3a62516770d3eaf… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Jan 5 20:42:19 2010 +0100 winedbg: Update link. --- programs/winedbg/gdbproxy.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/winedbg/gdbproxy.c b/programs/winedbg/gdbproxy.c index ceae777..c1dac01 100644 --- a/programs/winedbg/gdbproxy.c +++ b/programs/winedbg/gdbproxy.c @@ -21,7 +21,7 @@ */ /* Protocol specification can be found here: - * http://sources.redhat.com/gdb/onlinedocs/gdb_32.html + * http://sources.redhat.com/gdb/onlinedocs/gdb/Maintenance-Commands.html */ #include "config.h"

1 0

André Hentschel : dbghelp: Fix typo in url.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 2f5ba0d8c0996430e35188ab8e6e0f873d95ea63 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2f5ba0d8c0996430e35188ab8… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Jan 5 20:42:30 2010 +0100 dbghelp: Fix typo in url. --- dlls/dbghelp/stabs.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/dbghelp/stabs.c b/dlls/dbghelp/stabs.c index bc5b674..837e213 100644 --- a/dlls/dbghelp/stabs.c +++ b/dlls/dbghelp/stabs.c @@ -26,7 +26,7 @@ * The "stabs" debug format * by Julia Menapace, Jim Kingdon, David Mackenzie * of Cygnus Support - * available (hopefully) from http:\\sources.redhat.com\gdb\onlinedocs + * available (hopefully) from http://sources.redhat.com/gdb/onlinedocs */ #include "config.h"

1 0

André Hentschel : msi/tests: Remove dead url.
by Alexandre Julliard Jan. 6, 2010

Jan. 6, 2010

Module: wine Branch: master Commit: 2e339c471e82029263fab04f19aa9b5179c5d471 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2e339c471e82029263fab04f1… Author: André Hentschel <nerv(a)dawncrow.de> Date: Tue Jan 5 20:43:09 2010 +0100 msi/tests: Remove dead url. --- dlls/msi/tests/record.c | 1 - 1 files changed, 0 insertions(+), 1 deletions(-) diff --git a/dlls/msi/tests/record.c b/dlls/msi/tests/record.c index 369c855..0b80779 100644 --- a/dlls/msi/tests/record.c +++ b/dlls/msi/tests/record.c @@ -334,7 +334,6 @@ static void test_msirecord(void) ok(r == ERROR_SUCCESS, "failed to add stream to record\n"); r = MsiRecordReadStream(h, 1, buf, NULL); ok(r == ERROR_INVALID_PARAMETER, "should return error\n"); - /* http://test.winehq.org/data/200503181000/98_jmelgarejo98casa/msi:record.txt */ DeleteFile(filename); /* Windows 98 doesn't like this at all, so don't check return. */ r = MsiRecordReadStream(h, 1, NULL, NULL); ok(r == ERROR_INVALID_PARAMETER, "should return error\n");

1 0

Alistair Leslie-Hughes : gdiplus: Implement GdipIsVisibleClipEmpty.
by Alexandre Julliard Jan. 5, 2010

Jan. 5, 2010

Module: wine Branch: master Commit: 03e31f9b86dc645574ca2ee21038d04997093f54 URL: http://source.winehq.org/git/wine.git/?a=commit;h=03e31f9b86dc645574ca2ee21… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Wed Dec 23 13:53:27 2009 +1100 gdiplus: Implement GdipIsVisibleClipEmpty. --- dlls/gdiplus/gdiplus.spec | 2 +- dlls/gdiplus/graphics.c | 23 +++++++++++++++++++++++ 2 files changed, 24 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index aa64a3c..7d6081d 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -424,7 +424,7 @@ @ stdcall GdipIsOutlineVisiblePathPoint(ptr long long ptr ptr ptr) @ stdcall GdipIsOutlineVisiblePathPointI(ptr long long ptr ptr ptr) @ stdcall GdipIsStyleAvailable(ptr long ptr) -@ stub GdipIsVisibleClipEmpty +@ stdcall GdipIsVisibleClipEmpty(ptr ptr) @ stdcall GdipIsVisiblePathPoint(ptr long long ptr ptr) @ stdcall GdipIsVisiblePathPointI(ptr long long ptr ptr) @ stdcall GdipIsVisiblePoint(ptr long long ptr) diff --git a/dlls/gdiplus/graphics.c b/dlls/gdiplus/graphics.c index 67b0ff9..39ab18c 100644 --- a/dlls/gdiplus/graphics.c +++ b/dlls/gdiplus/graphics.c @@ -4359,3 +4359,26 @@ GpStatus WINGDIPAPI GdipRecordMetafileI(HDC hdc, EmfType type, GDIPCONST GpRect FIXME("(%p %d %p %d %p %p): stub\n", hdc, type, frameRect, frameUnit, desc, metafile); return NotImplemented; } + +/***************************************************************************** + * GdipIsVisibleClipEmpty [GDIPLUS.@] + */ +GpStatus WINGDIPAPI GdipIsVisibleClipEmpty(GpGraphics *graphics, BOOL *res) +{ + GpStatus stat; + GpRegion* rgn; + + TRACE("(%p, %p)\n", graphics, res); + + if((stat = GdipCreateRegion(&rgn)) != Ok) + return stat; + + if((stat = get_visible_clip_region(graphics, rgn)) != Ok) + goto cleanup; + + stat = GdipIsEmptyRegion(rgn, graphics, res); + +cleanup: + GdipDeleteRegion(rgn); + return stat; +}

1 0