wine-commits October 2014

wine-commits@winehq.org

2 participants
397 discussions

Bruno Jesus : comctl32/tests: Remove a dead assignment from dpa.c ( Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 2c5f05b860d0f69b249746523374851d90ea5e51 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c5f05b860d0f69b249746523… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Fri Oct 31 00:32:04 2014 -0200 comctl32/tests: Remove a dead assignment from dpa.c (Cppcheck). --- dlls/comctl32/tests/dpa.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comctl32/tests/dpa.c b/dlls/comctl32/tests/dpa.c index 644f3aa..4eb32a4 100644 --- a/dlls/comctl32/tests/dpa.c +++ b/dlls/comctl32/tests/dpa.c @@ -271,7 +271,7 @@ static void test_dpa(void) ok(rc, "dw2=0x%x\n", dw2); /* Clone into an old DPA */ - p = NULL; SetLastError(ERROR_SUCCESS); + SetLastError(ERROR_SUCCESS); p = pDPA_Clone(dpa, dpa3); ok(p == dpa3, "p=%p\n", p); rc=CheckDPA(dpa3, 0x123456, &dw3);

1 0

Bruno Jesus : comctl32/tests: Remove dead assignment in comboex.c ( Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 2ccd55e56f9c0d8482ca8fe5d820b95f67647b90 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2ccd55e56f9c0d8482ca8fe5d… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Fri Oct 31 00:24:38 2014 -0200 comctl32/tests: Remove dead assignment in comboex.c (Cppcheck). --- dlls/comctl32/tests/comboex.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comctl32/tests/comboex.c b/dlls/comctl32/tests/comboex.c index af01839..faa32a3 100644 --- a/dlls/comctl32/tests/comboex.c +++ b/dlls/comctl32/tests/comboex.c @@ -127,7 +127,7 @@ static HWND subclass_editbox(HWND hwndComboEx) static void test_comboboxex(void) { HWND myHwnd = 0; - LONG res = -1; + LONG res; COMBOBOXEXITEMA cbexItem; static const char *first_item = "First Item", *second_item = "Second Item",

1 0

Bruno Jesus : comctl32: Remove dead assignment in tooltips.c (Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: a54c187886353bd7c6bcc6d43b85e1101a0627f4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=a54c187886353bd7c6bcc6d43… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Fri Oct 31 00:17:34 2014 -0200 comctl32: Remove dead assignment in tooltips.c (Cppcheck). --- dlls/comctl32/tooltips.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/comctl32/tooltips.c b/dlls/comctl32/tooltips.c index f74ea65..7564c3f 100644 --- a/dlls/comctl32/tooltips.c +++ b/dlls/comctl32/tooltips.c @@ -234,7 +234,7 @@ TOOLTIPS_customdraw_fill(const TOOLTIPS_INFO *infoPtr, NMTTCUSTOMDRAW *lpnmttcd, static inline DWORD TOOLTIPS_notify_customdraw (DWORD dwDrawStage, NMTTCUSTOMDRAW *lpnmttcd) { - LRESULT result = CDRF_DODEFAULT; + LRESULT result; lpnmttcd->nmcd.dwDrawStage = dwDrawStage; TRACE("Notifying stage %d, flags %x, id %x\n", lpnmttcd->nmcd.dwDrawStage,

1 0

Bruno Jesus : comctl32: Avoid magic numbers.
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: f70167df587a4af9954d0733fdd3e8d0b24e0289 URL: http://source.winehq.org/git/wine.git/?a=commit;h=f70167df587a4af9954d0733f… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Fri Oct 31 00:12:23 2014 -0200 comctl32: Avoid magic numbers. --- dlls/comctl32/string.c | 8 +++----- 1 file changed, 3 insertions(+), 5 deletions(-) diff --git a/dlls/comctl32/string.c b/dlls/comctl32/string.c index a9544fa..4c52f18 100644 --- a/dlls/comctl32/string.c +++ b/dlls/comctl32/string.c @@ -855,16 +855,15 @@ int WINAPI StrCSpnW(LPCWSTR lpszStr, LPCWSTR lpszMatch) BOOL WINAPI IntlStrEqWorkerA(BOOL bCase, LPCSTR lpszStr, LPCSTR lpszComp, int iLen) { - DWORD dwFlags = LOCALE_USE_CP_ACP; + DWORD dwFlags; int iRet; TRACE("(%d,%s,%s,%d)\n", bCase, debugstr_a(lpszStr), debugstr_a(lpszComp), iLen); /* FIXME: This flag is undocumented and unknown by our CompareString. - * We need a define for it. */ - dwFlags = 0x10000000; + dwFlags = LOCALE_RETURN_GENITIVE_NAMES; if (!bCase) dwFlags |= NORM_IGNORECASE; iRet = CompareStringA(GetThreadLocale(), @@ -891,9 +890,8 @@ BOOL WINAPI IntlStrEqWorkerW(BOOL bCase, LPCWSTR lpszStr, LPCWSTR lpszComp, debugstr_w(lpszStr),debugstr_w(lpszComp), iLen); /* FIXME: This flag is undocumented and unknown by our CompareString. - * We need a define for it. */ - dwFlags = 0x10000000; + dwFlags = LOCALE_RETURN_GENITIVE_NAMES; if (!bCase) dwFlags |= NORM_IGNORECASE; iRet = CompareStringW(GetThreadLocale(),

1 0

Bruno Jesus : comctl32: Remove a dead assignment in comctl32undoc.c ( Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 197ea35c5a0f140d4cedd24c754f9eb83afc1ac0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=197ea35c5a0f140d4cedd24c7… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Thu Oct 30 23:41:01 2014 -0200 comctl32: Remove a dead assignment in comctl32undoc.c (Cppcheck). --- dlls/comctl32/comctl32undoc.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/comctl32/comctl32undoc.c b/dlls/comctl32/comctl32undoc.c index f20857c..97bbf1a 100644 --- a/dlls/comctl32/comctl32undoc.c +++ b/dlls/comctl32/comctl32undoc.c @@ -690,8 +690,8 @@ static HANDLE create_mru_list(LPWINEMRULIST mp) /* get values from key 'MRUList' */ if (newkey) { datasize = (mp->extview.uMax + 1) * sizeof(WCHAR); - if((err=RegQueryValueExW( newkey, strMRUList, 0, &type, - (LPBYTE)mp->realMRU, &datasize))) { + if (RegQueryValueExW( newkey, strMRUList, 0, &type, + (LPBYTE)mp->realMRU, &datasize)) { /* not present - set size to 1 (will become 0 later) */ datasize = 1; *mp->realMRU = 0;

1 0

Alistair Leslie-Hughes : dplayx: Fix some crashes and wine_todo's.
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 60db3a4a86ad3a19682dfb483a4c4f37dff1ea29 URL: http://source.winehq.org/git/wine.git/?a=commit;h=60db3a4a86ad3a19682dfb483… Author: Alistair Leslie-Hughes <leslie_alistair(a)hotmail.com> Date: Thu Sep 25 15:05:40 2014 +1000 dplayx: Fix some crashes and wine_todo's. --- dlls/dplayx/tests/dplayx.c | 26 +++++++++++++++++++------- 1 file changed, 19 insertions(+), 7 deletions(-) diff --git a/dlls/dplayx/tests/dplayx.c b/dlls/dplayx/tests/dplayx.c index 7b6c0c0..fb6646d 100644 --- a/dlls/dplayx/tests/dplayx.c +++ b/dlls/dplayx/tests/dplayx.c @@ -685,7 +685,7 @@ static void init_TCPIP_provider( IDirectPlay4 *pDP, LPCSTR strIPAddressString, W } hr = IDirectPlayX_InitializeConnection( pDP, pAddress, 0 ); - todo_wine checkHR( DP_OK, hr ); + checkHR( DP_OK, hr ); HeapFree( GetProcessHeap(), 0, pAddress ); @@ -743,7 +743,7 @@ static void test_DirectPlayCreate(void) if ( hr == DP_OK ) IDirectPlayX_Release( pDP ); hr = DirectPlayCreate( (LPGUID) &DPSPGUID_TCPIP, &pDP, NULL ); - todo_wine checkHR( DP_OK, hr ); + checkHR( DP_OK, hr ); if ( hr == DP_OK ) IDirectPlayX_Release( pDP ); @@ -915,9 +915,9 @@ static BOOL CALLBACK EnumConnections_cb2( LPCGUID lpguidSP, if( IsEqualGUID(lpguidSP, &DPSPGUID_TCPIP) ) { hr = IDirectPlayX_InitializeConnection( pDP, lpConnection, 0 ); - todo_wine checkHR( DP_OK, hr ); + checkHR( DP_OK, hr ); hr = IDirectPlayX_InitializeConnection( pDP, lpConnection, 0 ); - todo_wine checkHR( DPERR_ALREADYINITIALIZED, hr ); + checkHR( DPERR_ALREADYINITIALIZED, hr ); } return TRUE; @@ -976,7 +976,7 @@ static void test_GetCaps(void) { hr = IDirectPlayX_GetCaps( pDP, &dpcaps, dwFlags ); - todo_wine checkHR( DP_OK, hr ); + checkHR( DP_OK, hr ); if ( hr == DP_OK ) @@ -1085,7 +1085,7 @@ static void test_Open(void) /* Uninitialized dpsd */ hr = IDirectPlayX_Open( pDP_server, &dpsd_server, DPOPEN_CREATE ); - todo_wine checkHR( DPERR_INVALIDPARAMS, hr ); + checkHR( DPERR_INVALIDPARAMS, hr ); dpsd_server.dwSize = sizeof(DPSESSIONDESC2); @@ -1095,7 +1095,7 @@ static void test_Open(void) /* Regular operation */ hr = IDirectPlayX_Open( pDP_server, &dpsd_server, DPOPEN_CREATE ); - todo_wine checkHR( DP_OK, hr ); + checkHR( DP_OK, hr ); /* Opening twice */ hr = IDirectPlayX_Open( pDP_server, &dpsd_server, DPOPEN_CREATE ); @@ -1698,12 +1698,16 @@ static void test_SessionDesc(void) checkHR( DPERR_INVALIDPARAMS, hr ); hr = IDirectPlayX_GetSessionDesc( pDP[0], NULL, NULL ); checkHR( DPERR_INVALIDPARAM, hr ); +if(0) +{ + /* Crashes under Win7 */ hr = IDirectPlayX_GetSessionDesc( pDP[0], lpData[0], NULL ); checkHR( DPERR_INVALIDPARAM, hr ); dwDataSize=-1; hr = IDirectPlayX_GetSessionDesc( pDP[0], lpData[0], &dwDataSize ); checkHR( DPERR_INVALIDPARAMS, hr ); check( -1, dwDataSize ); +} /* Get: Insufficient buffer size */ dwDataSize=0; @@ -2498,10 +2502,14 @@ static void test_PlayerName(void) checkHR( DPERR_INVALIDPLAYER, hr ); check( 1024, dwDataSize ); +if(0) +{ + /* Crashes under Win7 */ dwDataSize = -1; hr = IDirectPlayX_GetPlayerName( pDP[0], dpid[0], lpData, &dwDataSize ); checkHR( DPERR_INVALIDPARAMS, hr ); check( -1, dwDataSize ); +} hr = IDirectPlayX_GetPlayerName( pDP[0], dpid[0], lpData, NULL ); checkHR( DPERR_INVALIDPARAMS, hr ); @@ -5789,6 +5797,9 @@ static void test_GetMessageQueue(void) check( -1, dwNumBytes ); /* - Remote players */ +if(0) +{ + /* Crash under Win7 */ dwNumMsgs = dwNumBytes = -1; hr = IDirectPlayX_GetMessageQueue( pDP[0], 0, dpid[3], DPMESSAGEQUEUE_RECEIVE, @@ -5796,6 +5807,7 @@ static void test_GetMessageQueue(void) checkHR( DPERR_INVALIDPLAYER, hr ); /* Player 3 is remote */ check( -1, dwNumMsgs ); check( -1, dwNumBytes ); +} dwNumMsgs = dwNumBytes = -1; hr = IDirectPlayX_GetMessageQueue( pDP[0], dpid[3], 0,

1 0

Bruno Jesus : atl: Remove dead assignment.
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 36c331f2e95d69499ee734523d85c285bf58ff33 URL: http://source.winehq.org/git/wine.git/?a=commit;h=36c331f2e95d69499ee734523… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Thu Oct 30 23:22:55 2014 -0200 atl: Remove dead assignment. --- dlls/atl/registrar.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/atl/registrar.c b/dlls/atl/registrar.c index 32e207e..96fcb0e 100644 --- a/dlls/atl/registrar.c +++ b/dlls/atl/registrar.c @@ -380,7 +380,7 @@ static HRESULT do_process_root_key(LPCOLESTR data, BOOL do_register) { LPCOLESTR iter = data; strbuf buf; - HRESULT hres = S_OK; + HRESULT hres; unsigned int i; strbuf_init(&buf);

1 0

Bruno Jesus : atl: Remove dead assignment.
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: 08f3edccde763524e55e26a978da9a05d1219d64 URL: http://source.winehq.org/git/wine.git/?a=commit;h=08f3edccde763524e55e26a97… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Thu Oct 30 23:20:35 2014 -0200 atl: Remove dead assignment. --- dlls/atl/registrar.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/atl/registrar.c b/dlls/atl/registrar.c index 90c5b5b..32e207e 100644 --- a/dlls/atl/registrar.c +++ b/dlls/atl/registrar.c @@ -184,7 +184,7 @@ static HRESULT do_preprocess(const Registrar *This, LPCOLESTR data, strbuf *buf) static HRESULT do_process_key(LPCOLESTR *pstr, HKEY parent_key, strbuf *buf, BOOL do_register) { - LPCOLESTR iter = *pstr; + LPCOLESTR iter; HRESULT hres; LONG lres; HKEY hkey = 0;

1 0

Bruno Jesus : advpack/tests: Remove dead assignment (Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: c1b2fc3c4f89efd753abf17272600c9ae7c5fbe5 URL: http://source.winehq.org/git/wine.git/?a=commit;h=c1b2fc3c4f89efd753abf1727… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Thu Oct 30 23:16:14 2014 -0200 advpack/tests: Remove dead assignment (Cppcheck). --- dlls/advpack/tests/advpack.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/advpack/tests/advpack.c b/dlls/advpack/tests/advpack.c index f3e70d7..5b9ed42 100644 --- a/dlls/advpack/tests/advpack.c +++ b/dlls/advpack/tests/advpack.c @@ -187,7 +187,6 @@ static void delnode_test(void) CloseHandle(hn); hr = pDelNode(lstrcatA(currDir, "\\DelNodeTestDir"), 0); ok (hr == S_OK, "DelNode failed deleting a directory containing multiple files\n"); - currDir[currDirLen] = '\0'; } static void append_str(char **str, const char *data, ...)

1 0

Bruno Jesus : advapi32/tests: Remove dead assignment (Cppcheck).
by Alexandre Julliard 31 Oct '14

31 Oct '14

Module: wine Branch: master Commit: d3b406ecce238e43e042bf90a2e02cfc2eae070f URL: http://source.winehq.org/git/wine.git/?a=commit;h=d3b406ecce238e43e042bf90a… Author: Bruno Jesus <00cpxxx(a)gmail.com> Date: Thu Oct 30 23:13:04 2014 -0200 advapi32/tests: Remove dead assignment (Cppcheck). --- dlls/advapi32/tests/security.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/advapi32/tests/security.c b/dlls/advapi32/tests/security.c index b44496a..04a88ae 100644 --- a/dlls/advapi32/tests/security.c +++ b/dlls/advapi32/tests/security.c @@ -920,7 +920,6 @@ cleanup: rc &= ~(FILE_ATTRIBUTE_NOT_CONTENT_INDEXED|FILE_ATTRIBUTE_COMPRESSED); ok(rc == FILE_ATTRIBUTE_ARCHIVE, "expected FILE_ATTRIBUTE_ARCHIVE got %#x\n", rc); - retSize = 0xdeadbeef; rc = GetFileSecurityA(file, OWNER_SECURITY_INFORMATION|GROUP_SECURITY_INFORMATION|DACL_SECURITY_INFORMATION, NULL, 0, &sdSize); ok(!rc, "GetFileSecurity should fail\n");

1 0

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 October 2014