March 2009 - Wine-Commits - list.winehq.org

Christian Costa : amstream: Implement IAMMultiMediaStreamImpl_GetFilterGraph.
by Alexandre Julliard March 26, 2009

March 26, 2009

Module: wine Branch: master Commit: 8469689a71a02c21942f7caefe686de3d7505226 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8469689a71a02c21942f7caef… Author: Christian Costa <titan.costa(a)wanadoo.fr> Date: Wed Mar 25 22:55:03 2009 +0100 amstream: Implement IAMMultiMediaStreamImpl_GetFilterGraph. --- dlls/amstream/amstream.c | 12 ++++++++++-- 1 files changed, 10 insertions(+), 2 deletions(-) diff --git a/dlls/amstream/amstream.c b/dlls/amstream/amstream.c index 9e4d45a..6c544f6 100644 --- a/dlls/amstream/amstream.c +++ b/dlls/amstream/amstream.c @@ -238,9 +238,17 @@ static HRESULT WINAPI IAMMultiMediaStreamImpl_GetFilterGraph(IAMMultiMediaStream { IAMMultiMediaStreamImpl *This = (IAMMultiMediaStreamImpl *)iface; - FIXME("(%p/%p)->(%p) stub!\n", This, iface, ppGraphBuilder); + TRACE("(%p/%p)->(%p)\n", This, iface, ppGraphBuilder); - return E_NOTIMPL; + if (!ppGraphBuilder) + return E_POINTER; + + if (This->pFilterGraph) + return IFilterGraph_QueryInterface(This->pFilterGraph, &IID_IGraphBuilder, (void**)ppGraphBuilder); + else + *ppGraphBuilder = NULL; + + return S_OK; } static HRESULT WINAPI IAMMultiMediaStreamImpl_GetFilter(IAMMultiMediaStream* iface, IMediaStreamFilter** ppFilter)

1 0

Alexandre Julliard : ntdll: Get rid of the no longer used pthread emulation .
by Alexandre Julliard March 26, 2009

March 26, 2009

Module: wine Branch: master Commit: c4567214356c61e7d5fd2d1635df91caed681030 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c4567214356c61e7d5fd2d163… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Mar 26 13:20:04 2009 +0100 ntdll: Get rid of the no longer used pthread emulation. --- dlls/ntdll/Makefile.in | 1 - dlls/ntdll/ntdll_misc.h | 1 - dlls/ntdll/pthread.c | 599 ----------------------------------------------- dlls/ntdll/thread.c | 1 - 4 files changed, 0 insertions(+), 602 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c4567214356c61e7d5fd2…

1 0

Nikolay Sivov : comctl32/tab: Handle state mask in TCM_[G,S]ETITEM.
by Alexandre Julliard March 26, 2009

March 26, 2009

Module: wine Branch: master Commit: db60f69cba70795a43e557b0c96890a2f2e6d8e5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=db60f69cba70795a43e557b0c… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Mar 25 17:39:11 2009 -0400 comctl32/tab: Handle state mask in TCM_[G,S]ETITEM. --- dlls/comctl32/tab.c | 5 +++-- dlls/comctl32/tests/tab.c | 23 +++++++++++++++++++++++ 2 files changed, 26 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/tab.c b/dlls/comctl32/tab.c index 800857f..ac81016 100644 --- a/dlls/comctl32/tab.c +++ b/dlls/comctl32/tab.c @@ -2719,7 +2719,8 @@ TAB_SetItemT (TAB_INFO *infoPtr, INT iItem, LPTCITEMW tabItem, BOOL bUnicode) FIXME("TCIF_RTLREADING\n"); if (tabItem->mask & TCIF_STATE) - wineItem->dwState = tabItem->dwState; + wineItem->dwState = (wineItem->dwState & ~tabItem->dwStateMask) | + ( tabItem->dwState & tabItem->dwStateMask); if (tabItem->mask & TCIF_TEXT) { @@ -2766,7 +2767,7 @@ TAB_GetItemT (TAB_INFO *infoPtr, INT iItem, LPTCITEMW tabItem, BOOL bUnicode) FIXME("TCIF_RTLREADING\n"); if (tabItem->mask & TCIF_STATE) - tabItem->dwState = wineItem->dwState; + tabItem->dwState = wineItem->dwState & tabItem->dwStateMask; if (tabItem->mask & TCIF_TEXT) { diff --git a/dlls/comctl32/tests/tab.c b/dlls/comctl32/tests/tab.c index d24d097..5f5e4b9 100644 --- a/dlls/comctl32/tests/tab.c +++ b/dlls/comctl32/tests/tab.c @@ -831,6 +831,29 @@ static void test_getters_setters(HWND parent_wnd, INT nTabs) ok_sequence(sequences, TAB_SEQ_INDEX, getset_item_seq, "Getset item test sequence", FALSE); ok_sequence(sequences, PARENT_SEQ_INDEX, empty_sequence, "Getset item test parent sequence", FALSE); + + /* TCIS_BUTTONPRESSED doesn't depend on tab style */ + memset(&tcItem, 0, sizeof(tcItem)); + tcItem.mask = TCIF_STATE; + tcItem.dwStateMask = TCIS_BUTTONPRESSED; + tcItem.dwState = TCIS_BUTTONPRESSED; + ok ( SendMessage(hTab, TCM_SETITEM, 0, (LPARAM) &tcItem), "Setting new item failed.\n"); + tcItem.dwState = 0; + ok ( SendMessage(hTab, TCM_GETITEM, 0, (LPARAM) &tcItem), "Getting item failed.\n"); + ok (tcItem.dwState == TCIS_BUTTONPRESSED, "TCIS_BUTTONPRESSED should be set.\n"); + /* next highlight item, test that dwStateMask actually masks */ + tcItem.mask = TCIF_STATE; + tcItem.dwStateMask = TCIS_HIGHLIGHTED; + tcItem.dwState = TCIS_HIGHLIGHTED; + ok ( SendMessage(hTab, TCM_SETITEM, 0, (LPARAM) &tcItem), "Setting new item failed.\n"); + tcItem.dwState = 0; + ok ( SendMessage(hTab, TCM_GETITEM, 0, (LPARAM) &tcItem), "Getting item failed.\n"); + ok (tcItem.dwState == TCIS_HIGHLIGHTED, "TCIS_HIGHLIGHTED should be set.\n"); + tcItem.mask = TCIF_STATE; + tcItem.dwStateMask = TCIS_BUTTONPRESSED; + tcItem.dwState = 0; + ok ( SendMessage(hTab, TCM_GETITEM, 0, (LPARAM) &tcItem), "Getting item failed.\n"); + ok (tcItem.dwState == TCIS_BUTTONPRESSED, "TCIS_BUTTONPRESSED should be set.\n"); } /* Testing GetSet ToolTip */

1 0

Nikolay Sivov : comctl32/tab: Draw tab button properly on TCIS_BUTTONPRESSED state.
by Alexandre Julliard March 26, 2009

March 26, 2009

Module: wine Branch: master Commit: 751732e6949808d7ca240a56513f6d1320bce132 URL: http://source.winehq.org/git/wine.git/?a=commit;h=751732e6949808d7ca240a565… Author: Nikolay Sivov <bunglehead(a)gmail.com> Date: Wed Mar 25 16:34:48 2009 -0400 comctl32/tab: Draw tab button properly on TCIS_BUTTONPRESSED state. --- dlls/comctl32/tab.c | 9 +++++++-- 1 files changed, 7 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/tab.c b/dlls/comctl32/tab.c index 3a0e477..800857f 100644 --- a/dlls/comctl32/tab.c +++ b/dlls/comctl32/tab.c @@ -1962,8 +1962,13 @@ static void TAB_DrawItem(const TAB_INFO *infoPtr, HDC hdc, INT iItem) } else /* ! selected */ { - if (!(lStyle & TCS_FLATBUTTONS)) - DrawEdge(hdc, &r, EDGE_RAISED, BF_SOFT|BF_RECT); + DWORD state = infoPtr->items[iItem].dwState; + + if (state & TCIS_BUTTONPRESSED) + DrawEdge(hdc, &r, EDGE_SUNKEN, BF_SOFT|BF_RECT); + else + if (!(lStyle & TCS_FLATBUTTONS)) + DrawEdge(hdc, &r, EDGE_RAISED, BF_SOFT|BF_RECT); } } else /* !TCS_BUTTONS */

1 0

Michael Stefaniuc : d3dxof: Remove redundant NULL check before HeapFree ( Smatch).
by Alexandre Julliard March 26, 2009

March 26, 2009

Module: wine Branch: master Commit: 289dca2e0e5f132cb15917fa191a140487e15fb1 URL: http://source.winehq.org/git/wine.git/?a=commit;h=289dca2e0e5f132cb15917fa1… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Mar 25 23:02:42 2009 +0100 d3dxof: Remove redundant NULL check before HeapFree (Smatch). --- dlls/d3dxof/d3dxof.c | 3 +-- 1 files changed, 1 insertions(+), 2 deletions(-) diff --git a/dlls/d3dxof/d3dxof.c b/dlls/d3dxof/d3dxof.c index 6f9b266..5d42d53 100644 --- a/dlls/d3dxof/d3dxof.c +++ b/dlls/d3dxof/d3dxof.c @@ -1100,9 +1100,8 @@ static HRESULT WINAPI IDirectXFileEnumObjectImpl_GetNextDataObject(IDirectXFileE error: HeapFree(GetProcessHeap(), 0, This->buf.pxo_tab); + HeapFree(GetProcessHeap(), 0, This->buf.pxo->pdata); HeapFree(GetProcessHeap(), 0, pstrings); - if (This->buf.pxo->pdata) - HeapFree(GetProcessHeap(), 0, This->buf.pxo->pdata); return hr; }

1 0

Huw Davies : ole32: Move the code to set the clipboard formats to a separate function.
by Alexandre Julliard March 25, 2009

March 25, 2009

Module: wine Branch: master Commit: 3383db61074d05bbf3d5e4d58e9cbacb6a4c54e2 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3383db61074d05bbf3d5e4d58… Author: Huw Davies <huw(a)codeweavers.com> Date: Tue Mar 24 13:49:24 2009 +0000 ole32: Move the code to set the clipboard formats to a separate function. --- dlls/ole32/clipboard.c | 87 +++++++++++++++++++++++------------------------ 1 files changed, 43 insertions(+), 44 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 4c6f35f..008e35b 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -1271,6 +1271,46 @@ static HWND OLEClipbrd_CreateWindow(void) return hwnd; } +/********************************************************************* + * set_clipboard_formats + * + * Enumerate all HGLOBAL formats supported by the source and make + * those formats available using delayed rendering using SetClipboardData. + * + * TODO: We need to additionally handle TYMED_IStorage and + * TYMED_IStream data by copying into global memory. + */ +static HRESULT set_clipboard_formats(IDataObject *data) +{ + HRESULT hr; + FORMATETC fmt; + IEnumFORMATETC *enum_fmt; + + hr = IDataObject_EnumFormatEtc(data, DATADIR_GET, &enum_fmt); + if(FAILED(hr)) return hr; + + while(IEnumFORMATETC_Next(enum_fmt, 1, &fmt, NULL) == S_OK) + { + if (fmt.tymed == TYMED_HGLOBAL) + { + char fmt_name[80]; + TRACE("(cfFormat=%d:%s)\n", fmt.cfFormat, + GetClipboardFormatNameA(fmt.cfFormat, fmt_name, sizeof(fmt_name)-1) ? fmt_name : ""); + + SetClipboardData(fmt.cfFormat, NULL); + } + } + + IEnumFORMATETC_Release(enum_fmt); + return S_OK; +} + +/********************************************************************* + * set_dataobject_format + * + * Windows creates a 'DataObject' clipboard format that contains the + * clipboard window's HWND or NULL if the Ole clipboard has been flushed. + */ static HRESULT set_dataobject_format(HWND hwnd) { HGLOBAL h = GlobalAlloc(GMEM_DDESHARE | GMEM_MOVEABLE, sizeof(hwnd)); @@ -1312,8 +1352,6 @@ static HRESULT set_dataobject_format(HWND hwnd) HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) { HRESULT hr = S_OK; - IEnumFORMATETC* penumFormatetc = NULL; - FORMATETC rgelt; BOOL bClipboardOpen = FALSE; struct oletls *info = COM_CurrentInfo(); @@ -1361,54 +1399,15 @@ HRESULT WINAPI OleSetClipboard(IDataObject* pDataObj) theOleClipboard->pIDataObjectSrc = NULL; } - /* - * AddRef the data object passed in and save its pointer. - * A NULL value indicates that the clipboard should be emptied. - */ + /* A NULL value indicates that the clipboard should be emptied. */ theOleClipboard->pIDataObjectSrc = pDataObj; if ( pDataObj ) { IDataObject_AddRef(theOleClipboard->pIDataObjectSrc); + hr = set_clipboard_formats(pDataObj); + if(FAILED(hr)) goto CLEANUP; } - /* - * Enumerate all HGLOBAL formats supported by the source and make - * those formats available using delayed rendering using SetClipboardData. - * Only global memory based data items may be made available to non-OLE - * applications via the standard Windows clipboard API. Data based on other - * mediums(non TYMED_HGLOBAL) can only be accessed via the Ole Clipboard API. - * - * TODO: Do we need to additionally handle TYMED_IStorage media by copying - * the storage into global memory? - */ - if ( pDataObj ) - { - if ( FAILED(hr = IDataObject_EnumFormatEtc( pDataObj, - DATADIR_GET, - &penumFormatetc ))) - { - HANDLE_ERROR( hr ); - } - - while ( S_OK == IEnumFORMATETC_Next(penumFormatetc, 1, &rgelt, NULL) ) - { - if ( rgelt.tymed == TYMED_HGLOBAL ) - { - CHAR szFmtName[80]; - TRACE("(cfFormat=%d:%s)\n", rgelt.cfFormat, - GetClipboardFormatNameA(rgelt.cfFormat, szFmtName, sizeof(szFmtName)-1) - ? szFmtName : ""); - - SetClipboardData( rgelt.cfFormat, NULL); - } - } - IEnumFORMATETC_Release(penumFormatetc); - } - - /* - * Windows additionally creates a new "DataObject" clipboard format - * and stores the clipboard window's HWND in it - */ hr = set_dataobject_format(theOleClipboard->hWndClipboard); CLEANUP:

1 0

Huw Davies : ole32/tests: Tests for the 'Ole Private Data' clipboard format .
by Alexandre Julliard March 25, 2009

March 25, 2009

Module: wine Branch: master Commit: b75caee2abbde87903f2a62edb8f215e037b9d33 URL: http://source.winehq.org/git/wine.git/?a=commit;h=b75caee2abbde87903f2a62ed… Author: Huw Davies <huw(a)codeweavers.com> Date: Wed Mar 25 14:18:51 2009 +0000 ole32/tests: Tests for the 'Ole Private Data' clipboard format. --- dlls/ole32/tests/clipboard.c | 233 +++++++++++++++++++++++++++++++++++++----- 1 files changed, 209 insertions(+), 24 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=b75caee2abbde87903f2a…

1 0

Alexandre Julliard : winebuild: Skip non-functions when building the undefined exports file.
by Alexandre Julliard March 25, 2009

March 25, 2009

Module: wine Branch: master Commit: cdad788d3315a8759c581d768eb4f3a59ee0a739 URL: http://source.winehq.org/git/wine.git/?a=commit;h=cdad788d3315a8759c581d768… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 25 13:27:27 2009 +0100 winebuild: Skip non-functions when building the undefined exports file. --- tools/winebuild/import.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/tools/winebuild/import.c b/tools/winebuild/import.c index 1cc3689..e782c99 100644 --- a/tools/winebuild/import.c +++ b/tools/winebuild/import.c @@ -510,7 +510,7 @@ static char *create_undef_symbols_file( DLLSPEC *spec ) for (i = 0; i < spec->nb_entry_points; i++) { ORDDEF *odp = &spec->entry_points[i]; - if (odp->type == TYPE_STUB) continue; + if (odp->type == TYPE_STUB || odp->type == TYPE_ABS) continue; if (odp->flags & FLAG_FORWARD) continue; fprintf( f, "\t%s %s\n", get_asm_ptr_keyword(), asm_name(odp->link_name) ); }

1 0

Rob Shearman : rpcrt4: Implement RPC over HTTP support.
by Alexandre Julliard March 25, 2009

March 25, 2009

Module: wine Branch: master Commit: 5f6ae94ad0163341c2cd79a89f29b2187367b5ba URL: http://source.winehq.org/git/wine.git/?a=commit;h=5f6ae94ad0163341c2cd79a89… Author: Rob Shearman <rob(a)codeweavers.com> Date: Wed Mar 18 11:13:19 2009 +0000 rpcrt4: Implement RPC over HTTP support. CodeWeavers did this work for supporting the optional HTTP connection method of Outlook 2003/2007 (must be running with Windows versions set to XP or higher to see this option). This was written before Microsoft publicly released a specification for the RPC over HTTP protocol and so was developed by examining traffic flowing between a Windows client and an IIS server. --- dlls/rpcrt4/Makefile.in | 2 +- dlls/rpcrt4/rpc_assoc.c | 1 + dlls/rpcrt4/rpc_assoc.h | 1 + dlls/rpcrt4/rpc_binding.h | 3 + dlls/rpcrt4/rpc_defs.h | 10 + dlls/rpcrt4/rpc_message.c | 332 ++++++++++++++- dlls/rpcrt4/rpc_message.h | 11 + dlls/rpcrt4/rpc_transport.c | 994 ++++++++++++++++++++++++++++++++++++++++++- 8 files changed, 1337 insertions(+), 17 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5f6ae94ad0163341c2cd7…

1 0

Alexandre Julliard : kernel32: Revert incorrect part of commit 7c3529f1cc6a3ce6f0a403c337926c8fc9fbfc57.
by Alexandre Julliard March 25, 2009

March 25, 2009

Module: wine Branch: master Commit: 79cd9183e810aab7f94ad5bbcf11406d27d16a94 URL: http://source.winehq.org/git/wine.git/?a=commit;h=79cd9183e810aab7f94ad5bbc… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Mar 25 14:19:02 2009 +0100 kernel32: Revert incorrect part of commit 7c3529f1cc6a3ce6f0a403c337926c8fc9fbfc57. --- dlls/kernel32/profile.c | 5 ++--- 1 files changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/profile.c b/dlls/kernel32/profile.c index 5d563a5..d44e819 100644 --- a/dlls/kernel32/profile.c +++ b/dlls/kernel32/profile.c @@ -208,9 +208,8 @@ static void PROFILE_Save( HANDLE hFile, const PROFILESECTION *section, ENCODING for (key = section->key; key; key = key->next) { - len += strlenW(key->name); - if (key->value && key->value[0]) len += strlenW(key->value); - len += 3; /* '=' and "\r\n" */ + len += strlenW(key->name) + 2; + if (key->value) len += strlenW(key->value) + 1; } buffer = HeapAlloc(GetProcessHeap(), 0, len * sizeof(WCHAR));

1 0