July 2008 - Wine-commits - list.winehq.org

Andrew Talbot : dinput: Remove unneeded address-of operators from array names.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 0993244b08df3edbc849fadd35b120c2c9526fe6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=0993244b08df3edbc849fadd3… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Wed Jul 9 20:36:48 2008 +0100 dinput: Remove unneeded address-of operators from array names. --- dlls/dinput/effect_linuxinput.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dinput/effect_linuxinput.c b/dlls/dinput/effect_linuxinput.c index 1fef4cd..9b1cb1b 100644 --- a/dlls/dinput/effect_linuxinput.c +++ b/dlls/dinput/effect_linuxinput.c @@ -875,7 +875,7 @@ HRESULT linuxinput_get_info_A( info->dwDynamicParams = info->dwStaticParams; /* yes, this is windows behavior (print the GUID_Name for name) */ - strcpy((char*)&(info->tszName), _dump_dinput_GUID(rguid)); + strcpy((char*)info->tszName, _dump_dinput_GUID(rguid)); return DI_OK; } @@ -910,7 +910,7 @@ HRESULT linuxinput_get_info_W( /* yes, this is windows behavior (print the GUID_Name for name) */ MultiByteToWideChar(CP_ACP, 0, _dump_dinput_GUID(rguid), -1, - (WCHAR*)&(info->tszName), MAX_PATH); + (WCHAR*)info->tszName, MAX_PATH); return DI_OK; }

1 0

Andrew Talbot : dmband: Remove unneeded address-of operators from array names.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 32a21e2e7165ef6849a06147e542dd6f14514bc0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=32a21e2e7165ef6849a06147e… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Wed Jul 9 20:36:51 2008 +0100 dmband: Remove unneeded address-of operators from array names. --- dlls/dmband/dmutils.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dmband/dmutils.c b/dlls/dmband/dmutils.c index a126513..bfec1f1 100644 --- a/dlls/dmband/dmutils.c +++ b/dlls/dmband/dmutils.c @@ -180,13 +180,13 @@ HRESULT IDirectMusicUtils_IPersistStream_ParseDescGeneric (DMUS_PRIVATE_CHUNK* p case DMUS_FOURCC_NAME_CHUNK: { TRACE_(dmfile)(": name chunk\n"); pDesc->dwValidData |= DMUS_OBJ_NAME; - IStream_Read (pStm, &pDesc->wszName, pChunk->dwSize, NULL); + IStream_Read (pStm, pDesc->wszName, pChunk->dwSize, NULL); break; } case DMUS_FOURCC_FILE_CHUNK: { TRACE_(dmfile)(": file name chunk\n"); pDesc->dwValidData |= DMUS_OBJ_FILENAME; - IStream_Read (pStm, &pDesc->wszFileName, pChunk->dwSize, NULL); + IStream_Read (pStm, pDesc->wszFileName, pChunk->dwSize, NULL); break; } case DMUS_FOURCC_VERSION_CHUNK: {

1 0

Andrew Talbot : dmime: Remove unneeded address-of operators from array names.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: f8f758f747da7a5ba11897b890f4a4e11c2e99ed URL: http://source.winehq.org/git/wine.git/?a=commit;h=f8f758f747da7a5ba11897b89… Author: Andrew Talbot <andrew.talbot(a)talbotville.com> Date: Wed Jul 9 20:36:54 2008 +0100 dmime: Remove unneeded address-of operators from array names. --- dlls/dmime/dmutils.c | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/dmime/dmutils.c b/dlls/dmime/dmutils.c index edcd121..1a822ba 100644 --- a/dlls/dmime/dmutils.c +++ b/dlls/dmime/dmutils.c @@ -78,13 +78,13 @@ HRESULT IDirectMusicUtils_IPersistStream_ParseDescGeneric (DMUS_PRIVATE_CHUNK* p case DMUS_FOURCC_NAME_CHUNK: { TRACE_(dmfile)(": name chunk\n"); pDesc->dwValidData |= DMUS_OBJ_NAME; - IStream_Read (pStm, &pDesc->wszName, pChunk->dwSize, NULL); + IStream_Read (pStm, pDesc->wszName, pChunk->dwSize, NULL); break; } case DMUS_FOURCC_FILE_CHUNK: { TRACE_(dmfile)(": file name chunk\n"); pDesc->dwValidData |= DMUS_OBJ_FILENAME; - IStream_Read (pStm, &pDesc->wszFileName, pChunk->dwSize, NULL); + IStream_Read (pStm, pDesc->wszFileName, pChunk->dwSize, NULL); break; } case DMUS_FOURCC_VERSION_CHUNK: {

1 0

Francois Gouget : msxml3/tests: Add a trailing '\n' to ok() calls.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: c3a952eb485597541736ba0e75880f9449b7e43a URL: http://source.winehq.org/git/wine.git/?a=commit;h=c3a952eb485597541736ba0e7… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Jul 10 00:23:43 2008 +0200 msxml3/tests: Add a trailing '\n' to ok() calls. --- dlls/msxml3/tests/domdoc.c | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/msxml3/tests/domdoc.c b/dlls/msxml3/tests/domdoc.c index f914e09..3745fca 100644 --- a/dlls/msxml3/tests/domdoc.c +++ b/dlls/msxml3/tests/domdoc.c @@ -1170,17 +1170,17 @@ todo_wine ok( r == E_INVALIDARG, "ret %08x\n", r ); r = IXMLDOMNodeList_get_item(list, 0, NULL); - ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x", r); + ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x\n", r); r = IXMLDOMNodeList_get_length(list, NULL); - ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x", r); + ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x\n", r); r = IXMLDOMNodeList_get_length( list, &count ); ok( r == S_OK, "get_length returns %08x\n", r ); ok( count == 4, "get_length got %ld\n", count ); r = IXMLDOMNodeList_nextNode(list, NULL); - ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x", r); + ok(r == E_INVALIDARG, "Exected E_INVALIDARG got %08x\n", r); r = IXMLDOMNodeList_nextNode( list, &node ); ok( r == S_OK, "nextNode returned wrong code\n");

1 0

Francois Gouget : gdiplus/tests: Spelling fix.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 4b1f6310d96c6da4e6a13d117fa4ed247f63c6ec URL: http://source.winehq.org/git/wine.git/?a=commit;h=4b1f6310d96c6da4e6a13d117… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Jul 10 00:24:46 2008 +0200 gdiplus/tests: Spelling fix. --- dlls/gdiplus/tests/font.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/gdiplus/tests/font.c b/dlls/gdiplus/tests/font.c index 8bf1289..96ed64c 100644 --- a/dlls/gdiplus/tests/font.c +++ b/dlls/gdiplus/tests/font.c @@ -25,7 +25,7 @@ #define expect(expected, got) ok(got == expected, "Expected %.8x, got %.8x\n", expected, got) static const WCHAR arial[] = {'A','r','i','a','l','\0'}; -static const WCHAR nonexistant[] = {'T','h','i','s','F','o','n','t','s','h','o','u','l','d','N','o','t','E','x','i','s','t','\0'}; +static const WCHAR nonexistent[] = {'T','h','i','s','F','o','n','t','s','h','o','u','l','d','N','o','t','E','x','i','s','t','\0'}; static const WCHAR MSSansSerif[] = {'M','S',' ','S','a','n','s',' ','S','e','r','i','f','\0'}; static const WCHAR MicrosoftSansSerif[] = {'M','i','c','r','o','s','o','f','t',' ','S','a','n','s',' ','S','e','r','i','f','\0'}; static const WCHAR TimesNewRoman[] = {'T','i','m','e','s',' ','N','e','w',' ','R','o','m','a','n','\0'}; @@ -48,7 +48,7 @@ static void test_createfont(void) UINT i; REAL size; - stat = GdipCreateFontFamilyFromName(nonexistant, NULL, &fontfamily); + stat = GdipCreateFontFamilyFromName(nonexistent, NULL, &fontfamily); expect (FontFamilyNotFound, stat); stat = GdipDeleteFont(font); expect (InvalidParameter, stat); @@ -161,10 +161,10 @@ static void test_fontfamily (void) /* FontFamily must be able to actually find the family. * If it can't, any subsequent calls should fail. */ - stat = GdipCreateFontFamilyFromName (nonexistant, NULL, &family); + stat = GdipCreateFontFamilyFromName (nonexistent, NULL, &family); expect (FontFamilyNotFound, stat); - ok ((lstrcmpiW(itsName, nonexistant) != 0), - "Expected a non-zero value for nonexistant font!\n"); + ok ((lstrcmpiW(itsName, nonexistent) != 0), + "Expected a non-zero value for nonexistent font!\n"); /* Bitmap fonts are not found */ todo_wine

1 0

Francois Gouget : rpcss: Add a trailing '\n' to a Wine trace.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 888cf5a4af96814d69056f63e41329466af6d2a6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=888cf5a4af96814d69056f63e… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Jul 10 00:26:43 2008 +0200 rpcss: Add a trailing '\n' to a Wine trace. --- programs/rpcss/epmp.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/programs/rpcss/epmp.c b/programs/rpcss/epmp.c index cf14be8..e731e06 100644 --- a/programs/rpcss/epmp.c +++ b/programs/rpcss/epmp.c @@ -86,7 +86,7 @@ static struct registered_ept_entry *find_ept_entry( void __RPC_USER ept_lookup_handle_t_rundown(ept_lookup_handle_t entry_handle) { - WINE_FIXME("%p", entry_handle); + WINE_FIXME("%p\n", entry_handle); } void ept_insert(handle_t h,

1 0

Michael Stefaniuc : quartz: Compare a file handle to INVALID_HANDLE_VALUE and not NULL.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 310b58994cae46289b0b685b276814d68fecec98 URL: http://source.winehq.org/git/wine.git/?a=commit;h=310b58994cae46289b0b685b2… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Jul 9 23:54:28 2008 +0200 quartz: Compare a file handle to INVALID_HANDLE_VALUE and not NULL. --- dlls/quartz/tests/avisplitter.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/quartz/tests/avisplitter.c b/dlls/quartz/tests/avisplitter.c index 3dd1615..d3cdaac 100644 --- a/dlls/quartz/tests/avisplitter.c +++ b/dlls/quartz/tests/avisplitter.c @@ -203,7 +203,7 @@ static void test_threads() file = CreateFileW(wfile, GENERIC_READ, FILE_SHARE_READ|FILE_SHARE_WRITE, NULL, OPEN_EXISTING, 0, NULL); - if (!file) + if (file == INVALID_HANDLE_VALUE) { skip("Could not read test file \"%s\", skipping test\n", afile); return;

1 0

Maarten Lankhorst : quartz: Fix deadlocks in dsound renderer.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: adcf93aeaf446d657685cee8976365980499fb69 URL: http://source.winehq.org/git/wine.git/?a=commit;h=adcf93aeaf446d657685cee89… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Jul 9 10:21:30 2008 -0700 quartz: Fix deadlocks in dsound renderer. --- dlls/quartz/dsoundrender.c | 10 +++++++++- 1 files changed, 9 insertions(+), 1 deletions(-) diff --git a/dlls/quartz/dsoundrender.c b/dlls/quartz/dsoundrender.c index 618f073..7a6ec3d 100644 --- a/dlls/quartz/dsoundrender.c +++ b/dlls/quartz/dsoundrender.c @@ -265,6 +265,7 @@ static HRESULT DSoundRender_Sample(LPVOID iface, IMediaSample * pSample) if (FAILED(hr)) { ERR("Cannot get pointer to sample data (%x)\n", hr); + LeaveCriticalSection(&This->csFilter); return hr; } @@ -281,6 +282,7 @@ static HRESULT DSoundRender_Sample(LPVOID iface, IMediaSample * pSample) if (IMediaSample_IsPreroll(pSample) == S_OK) { TRACE("Preroll!\n"); + LeaveCriticalSection(&This->csFilter); return S_OK; } @@ -298,6 +300,7 @@ static HRESULT DSoundRender_Sample(LPVOID iface, IMediaSample * pSample) if (This->state == State_Paused) { /* Assuming we return because of flushing */ + TRACE("Flushing\n"); LeaveCriticalSection(&This->csFilter); return S_OK; } @@ -887,7 +890,12 @@ static HRESULT WINAPI DSoundRender_InputPin_EndOfStream(IPin * iface) EnterCriticalSection(This->pin.pCritSec); TRACE("(%p/%p)->()\n", This, iface); - InputPin_EndOfStream(iface); + hr = InputPin_EndOfStream(iface); + if (hr != S_OK) + { + ERR("%08x\n", hr); + return hr; + } hr = IFilterGraph_QueryInterface(me->filterInfo.pGraph, &IID_IMediaEventSink, (LPVOID*)&pEventSink); if (SUCCEEDED(hr))

1 0

Francois Gouget : winhlp32: Remove spaces before '...'.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: bfc133f19b7c7d38bbdcc02803e2f3d832a87f63 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bfc133f19b7c7d38bbdcc0280… Author: Francois Gouget <fgouget(a)free.fr> Date: Thu Jul 10 00:20:50 2008 +0200 winhlp32: Remove spaces before '...'. --- programs/winhlp32/No.rc | 14 +++++++------- 1 files changed, 7 insertions(+), 7 deletions(-) diff --git a/programs/winhlp32/No.rc b/programs/winhlp32/No.rc index 2301c7c..b31df01 100644 --- a/programs/winhlp32/No.rc +++ b/programs/winhlp32/No.rc @@ -26,17 +26,17 @@ MAIN_MENU MENU LANGUAGE LANG_NORWEGIAN, SUBLANG_NORWEGIAN_BOKMAL MENUITEM "&�pne", MNID_FILE_OPEN MENUITEM SEPARATOR MENUITEM "Skriv &ut", MNID_FILE_PRINT - MENUITEM "Opp&sett av skriveren ...", MNID_FILE_SETUP + MENUITEM "Opp&sett av skriveren...", MNID_FILE_SETUP MENUITEM SEPARATOR MENUITEM "&Avslutt", MNID_FILE_EXIT } POPUP "R&ediger" { - MENUITEM "&Kopier ...", MNID_EDIT_COPYDLG + MENUITEM "&Kopier...", MNID_EDIT_COPYDLG MENUITEM SEPARATOR - MENUITEM "K&ommenter ...", MNID_EDIT_ANNOTATE + MENUITEM "K&ommenter...", MNID_EDIT_ANNOTATE } POPUP "&Bokmerke" { - MENUITEM "&Definer ...", MNID_BKMK_DEFINE + MENUITEM "&Definer...", MNID_BKMK_DEFINE } POPUP "&Innstillinger" { POPUP "Synlighet" @@ -58,7 +58,7 @@ MAIN_MENU MENU LANGUAGE LANG_NORWEGIAN, SUBLANG_NORWEGIAN_BOKMAL MENUITEM "&Hjelp til Hjelp", MNID_HELP_HELPON MENUITEM "All&tid �verst", MNID_HELP_HELPTOP MENUITEM SEPARATOR - MENUITEM "&Informasjon ...", MNID_HELP_ABOUT + MENUITEM "&Informasjon...", MNID_HELP_ABOUT #ifdef WINELIB MENUITEM "&Om Wine", MNID_HELP_WINE #endif @@ -98,9 +98,9 @@ CONTEXT_MENU MENU LANGUAGE LANG_NORWEGIAN, SUBLANG_NORWEGIAN_BOKMAL BEGIN POPUP "" BEGIN - MENUITEM "Kommentar ...", MNID_CTXT_ANNOTATE + MENUITEM "Kommentar...", MNID_CTXT_ANNOTATE MENUITEM "Kopier", MNID_CTXT_COPY - MENUITEM "Skriv ut ...", MNID_CTXT_PRINT + MENUITEM "Skriv ut...", MNID_CTXT_PRINT POPUP "Skriftst�rrelse" BEGIN MENUITEM "Liten", MNID_CTXT_FONTS_SMALL

1 0

Maarten Lankhorst : quartz: Always set current position again.
by Alexandre Julliard 10 Jul '08

10 Jul '08

Module: wine Branch: master Commit: 6748f073ed13eec690524391ff2cd1634764a0b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6748f073ed13eec690524391f… Author: Maarten Lankhorst <m.b.lankhorst(a)gmail.com> Date: Wed Jul 9 10:18:38 2008 -0700 quartz: Always set current position again. --- dlls/quartz/control.c | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/quartz/control.c b/dlls/quartz/control.c index 477f632..e719831 100644 --- a/dlls/quartz/control.c +++ b/dlls/quartz/control.c @@ -581,7 +581,7 @@ HRESULT WINAPI MediaSeekingImpl_SetPositions(IMediaSeeking * iface, LONGLONG * p llNewCurrent = Adjust(This->llCurrent, pCurrent, dwCurrentFlags); llNewStop = Adjust(This->llStop, pStop, dwStopFlags); - if (llNewCurrent != This->llCurrent) + if (pCurrent) bChangeCurrent = TRUE; if (llNewStop != This->llStop) bChangeStop = TRUE;

1 0