wine-commits February 2015

wine-commits@winehq.org

2 participants
411 discussions

Michael Stefaniuc : rsaenh: Remove an always false conditional subexpression (PVS-Studio).
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 17e565dbf91c3de8570d1581790aeb85aec5a971 URL: http://source.winehq.org/git/wine.git/?a=commit;h=17e565dbf91c3de8570d15817… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sat Feb 21 18:22:26 2015 +0100 rsaenh: Remove an always false conditional subexpression (PVS-Studio). --- dlls/rsaenh/mpi.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/rsaenh/mpi.c b/dlls/rsaenh/mpi.c index e964676..f75d89f 100644 --- a/dlls/rsaenh/mpi.c +++ b/dlls/rsaenh/mpi.c @@ -1955,8 +1955,8 @@ int mp_exptmod (const mp_int * G, const mp_int * X, mp_int * P, mp_int * Y) dr = 0; - /* if the modulus is odd or dr != 0 use the fast method */ - if (mp_isodd (P) == 1 || dr != 0) { + /* if the modulus is odd use the fast method */ + if (mp_isodd (P) == 1) { return mp_exptmod_fast (G, X, P, Y, dr); } else { /* otherwise use the generic Barrett reduction technique */

1 0

Michael Stefaniuc : oleaut32/tests: Remove an always true conditional subexpression (PVS-Studio).
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 724ee1d4fcf5807c490e60c2a99926de6a2ce34c URL: http://source.winehq.org/git/wine.git/?a=commit;h=724ee1d4fcf5807c490e60c2a… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sat Feb 21 18:06:16 2015 +0100 oleaut32/tests: Remove an always true conditional subexpression (PVS-Studio). --- dlls/oleaut32/tests/vartest.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/dlls/oleaut32/tests/vartest.c b/dlls/oleaut32/tests/vartest.c index 7ed4fd4..cf9e101 100644 --- a/dlls/oleaut32/tests/vartest.c +++ b/dlls/oleaut32/tests/vartest.c @@ -7548,7 +7548,7 @@ static void test_VarDiv(void) } /* Determine return type */ - if (!(rightvt == VT_EMPTY)) + if (rightvt != VT_EMPTY) { if (leftvt == VT_NULL || rightvt == VT_NULL) resvt = VT_NULL; @@ -7577,7 +7577,7 @@ static void test_VarDiv(void) else if (leftvt == VT_R4 || rightvt == VT_R4) resvt = VT_R4; } - else if (leftvt == VT_NULL && rightvt == VT_EMPTY) + else if (leftvt == VT_NULL) resvt = VT_NULL; else bFail = TRUE;

1 0

Michael Stefaniuc : mshtml/tests: Remove an always true conditional subexpression (PVS-Studio).
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 9dc35ac468f9c6e805a0eb1c57d043cabfe87a17 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9dc35ac468f9c6e805a0eb1c5… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sat Feb 21 18:04:34 2015 +0100 mshtml/tests: Remove an always true conditional subexpression (PVS-Studio). --- dlls/mshtml/tests/events.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/mshtml/tests/events.c b/dlls/mshtml/tests/events.c index 64c0556..8f4e15e 100644 --- a/dlls/mshtml/tests/events.c +++ b/dlls/mshtml/tests/events.c @@ -1412,7 +1412,7 @@ static void pump_msgs(BOOL *b) DispatchMessageW(&msg); } }else { - while(!b && PeekMessageW(&msg, NULL, 0, 0, PM_REMOVE)) { + while(PeekMessageW(&msg, NULL, 0, 0, PM_REMOVE)) { TranslateMessage(&msg); DispatchMessageW(&msg); }

1 0

Michael Stefaniuc : setupapi: RT_MANIFEST is already an int resource ( PVS-Studio).
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: d0dfa75ba4983ff594407d6415f5c822adb2c46f URL: http://source.winehq.org/git/wine.git/?a=commit;h=d0dfa75ba4983ff594407d641… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sat Feb 21 18:01:52 2015 +0100 setupapi: RT_MANIFEST is already an int resource (PVS-Studio). --- dlls/setupapi/fakedll.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/setupapi/fakedll.c b/dlls/setupapi/fakedll.c index 8e3dffb..b543ade 100644 --- a/dlls/setupapi/fakedll.c +++ b/dlls/setupapi/fakedll.c @@ -791,7 +791,7 @@ static void register_fake_dll( const WCHAR *name, const void *data, size_t size HMODULE module = (HMODULE)((ULONG_PTR)data | 1); HRSRC rsrc; - if ((rsrc = FindResourceW( module, manifestW, MAKEINTRESOURCEW(RT_MANIFEST) ))) + if ((rsrc = FindResourceW( module, manifestW, (LPWSTR)RT_MANIFEST ))) { char *manifest = LoadResource( module, rsrc ); register_manifest( name, manifest, SizeofResource( module, rsrc ), data, size );

1 0

Michael Stefaniuc : oledlg: RT_DIALOG is already an int resource ( PVS-Studio).
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: b9d5074aaab1d874c329ef848d4522c225743baf URL: http://source.winehq.org/git/wine.git/?a=commit;h=b9d5074aaab1d874c329ef848… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Sat Feb 21 18:00:53 2015 +0100 oledlg: RT_DIALOG is already an int resource (PVS-Studio). --- dlls/oledlg/pastespl.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/dlls/oledlg/pastespl.c b/dlls/oledlg/pastespl.c index f136438..4d31488 100644 --- a/dlls/oledlg/pastespl.c +++ b/dlls/oledlg/pastespl.c @@ -711,7 +711,7 @@ UINT WINAPI OleUIPasteSpecialW(LPOLEUIPASTESPECIALW ps) HRSRC hrsrc; if(name == NULL) return OLEUI_ERR_LPSZTEMPLATEINVALID; - hrsrc = FindResourceW(hInst, name, MAKEINTRESOURCEW(RT_DIALOG)); + hrsrc = FindResourceW(hInst, name, (LPWSTR)RT_DIALOG); if(!hrsrc) return OLEUI_ERR_FINDTEMPLATEFAILURE; dlg_templ = LoadResource(hInst, hrsrc); if(!dlg_templ) return OLEUI_ERR_LOADTEMPLATEFAILURE;

1 0

André Hentschel : oleaut32: Remove unused macros.
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: bedd444a3616fea54a060e07c27a137f29052fc9 URL: http://source.winehq.org/git/wine.git/?a=commit;h=bedd444a3616fea54a060e07c… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Feb 21 14:58:35 2015 +0100 oleaut32: Remove unused macros. --- dlls/oleaut32/varformat.c | 2 -- dlls/oleaut32/vartype.c | 2 -- 2 files changed, 4 deletions(-) diff --git a/dlls/oleaut32/varformat.c b/dlls/oleaut32/varformat.c index ec52a1e..27bedce 100644 --- a/dlls/oleaut32/varformat.c +++ b/dlls/oleaut32/varformat.c @@ -284,8 +284,6 @@ typedef struct tagFMT_DATE_HEADER #define FMT_NUM_ON_OFF 0x3F /* Convert to "On" or "Off" */ #define FMT_STR_COPY_SPACE 0x40 /* Copy len chars with space if no char */ #define FMT_STR_COPY_SKIP 0x41 /* Copy len chars or skip if no char */ -/* Wine additions */ -#define FMT_WINE_HOURS_12 0x81 /* Hours using 12 hour clock */ /* Named Formats and their tokenised values */ static const WCHAR szGeneralDate[] = { 'G','e','n','e','r','a','l',' ','D','a','t','e','\0' }; diff --git a/dlls/oleaut32/vartype.c b/dlls/oleaut32/vartype.c index e9b45ff..607d1a2 100644 --- a/dlls/oleaut32/vartype.c +++ b/dlls/oleaut32/vartype.c @@ -4148,8 +4148,6 @@ HRESULT WINAPI VarDecFromI4(LONG lIn, DECIMAL* pDecOut) return S_OK; } -#define LOCALE_EN_US (MAKELCID(MAKELANGID(LANG_ENGLISH,SUBLANG_ENGLISH_US),SORT_DEFAULT)) - /* internal representation of the value stored in a DECIMAL. The bytes are stored from LSB at index 0 to MSB at index 11 */

1 0

André Hentschel : strmbase: Remove unused macro.
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 72eefcb5c0a9d9c19407a279afc5810116ae78b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=72eefcb5c0a9d9c19407a279a… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Feb 21 14:58:31 2015 +0100 strmbase: Remove unused macro. --- dlls/strmbase/dllfunc.c | 1 - 1 file changed, 1 deletion(-) diff --git a/dlls/strmbase/dllfunc.c b/dlls/strmbase/dllfunc.c index 61ec283..195a409 100644 --- a/dlls/strmbase/dllfunc.c +++ b/dlls/strmbase/dllfunc.c @@ -51,7 +51,6 @@ static LONG server_locks = 0; /* * defines and constants */ -#define MAX_KEY_LEN 260 static const WCHAR clsid_keyname[] = {'C','L','S','I','D',0 }; static const WCHAR ips32_keyname[] = {'I','n','P','r','o','c','S','e','r','v','e','r','3','2',0};

1 0

André Hentschel : ole32: Remove unused macros.
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 8af03fef9a298df6140a012466d0e8a76dcd0b03 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8af03fef9a298df6140a01246… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Feb 21 14:58:33 2015 +0100 ole32: Remove unused macros. --- dlls/ole32/clipboard.c | 2 -- dlls/ole32/filelockbytes.c | 4 ---- dlls/ole32/ole2impl.c | 2 -- 3 files changed, 8 deletions(-) diff --git a/dlls/ole32/clipboard.c b/dlls/ole32/clipboard.c index 38df750..d40ee59 100644 --- a/dlls/ole32/clipboard.c +++ b/dlls/ole32/clipboard.c @@ -83,8 +83,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(ole); -#define HANDLE_ERROR(err) do { hr = err; TRACE("(HRESULT=%x)\n", (HRESULT)err); goto CLEANUP; } while (0) - /* Structure of 'Ole Private Data' clipboard format */ typedef struct { diff --git a/dlls/ole32/filelockbytes.c b/dlls/ole32/filelockbytes.c index 605b5df..7dc4843 100644 --- a/dlls/ole32/filelockbytes.c +++ b/dlls/ole32/filelockbytes.c @@ -65,10 +65,6 @@ static inline FileLockBytesImpl *impl_from_ILockBytes(ILockBytes *iface) * Prototypes for private methods */ -/* Note that this evaluates a and b multiple times, so don't - * pass expressions with side effects. */ -#define ROUND_UP(a, b) ((((a) + (b) - 1)/(b))*(b)) - /**************************************************************************** * GetProtectMode * diff --git a/dlls/ole32/ole2impl.c b/dlls/ole32/ole2impl.c index 5d92fec..152132b 100644 --- a/dlls/ole32/ole2impl.c +++ b/dlls/ole32/ole2impl.c @@ -36,8 +36,6 @@ WINE_DEFAULT_DEBUG_CHANNEL(ole); -#define MAX_CLIPFORMAT_NAME 80 - /****************************************************************************** * OleQueryCreateFromData [OLE32.@] *

1 0

André Hentschel : taskmgr: Remove dead code.
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 68d85cc7f8004c7d08584336ba24b18bf908ff9c URL: http://source.winehq.org/git/wine.git/?a=commit;h=68d85cc7f8004c7d08584336b… Author: André Hentschel <nerv(a)dawncrow.de> Date: Sat Feb 21 14:58:27 2015 +0100 taskmgr: Remove dead code. --- programs/taskmgr/taskmgr.c | 33 +-------------------------------- 1 file changed, 1 insertion(+), 32 deletions(-) diff --git a/programs/taskmgr/taskmgr.c b/programs/taskmgr/taskmgr.c index b876cbf..80dc81c 100644 --- a/programs/taskmgr/taskmgr.c +++ b/programs/taskmgr/taskmgr.c @@ -184,26 +184,13 @@ static BOOL OnCreate(HWND hWnd) GetClientRect(hWnd, &rc); nOldWidth = rc.right; nOldHeight = rc.bottom; - /* nOldStartX = rc.left; */ - /*nOldStartY = rc.top; */ - -#define PAGE_OFFSET_LEFT 17 -#define PAGE_OFFSET_TOP 72 -#define PAGE_OFFSET_WIDTH (PAGE_OFFSET_LEFT*2) -#define PAGE_OFFSET_HEIGHT (PAGE_OFFSET_TOP+32) if ((TaskManagerSettings.Left != 0) || (TaskManagerSettings.Top != 0) || (TaskManagerSettings.Right != 0) || (TaskManagerSettings.Bottom != 0)) - { MoveWindow(hWnd, TaskManagerSettings.Left, TaskManagerSettings.Top, TaskManagerSettings.Right - TaskManagerSettings.Left, TaskManagerSettings.Bottom - TaskManagerSettings.Top, TRUE); -#ifdef __GNUC__TEST__ - MoveWindow(hApplicationPage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); - MoveWindow(hProcessPage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); - MoveWindow(hPerformancePage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); -#endif - } + if (TaskManagerSettings.Maximized) ShowWindow(hWnd, SW_MAXIMIZE); @@ -297,19 +284,6 @@ static BOOL OnCreate(HWND hWnd) return TRUE; } -/* OnMove() - * This function handles all the moving events for the application - * It moves every child window that needs moving - */ -static void OnMove( UINT nType, int cx, int cy ) -{ -#ifdef __GNUC__TEST__ - MoveWindow(hApplicationPage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); - MoveWindow(hProcessPage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); - MoveWindow(hPerformancePage, TaskManagerSettings.Left + PAGE_OFFSET_LEFT, TaskManagerSettings.Top + PAGE_OFFSET_TOP, TaskManagerSettings.Right - TaskManagerSettings.Left - PAGE_OFFSET_WIDTH, TaskManagerSettings.Bottom - TaskManagerSettings.Top - PAGE_OFFSET_HEIGHT, FALSE); -#endif -} - /* OnSize() * This function handles all the sizing events for the application * It re-sizes every window, and child window that needs re-sizing @@ -1011,11 +985,6 @@ TaskManagerWndProc(HWND hDlg, UINT message, WPARAM wParam, LPARAM lParam) OnSize(wParam, LOWORD(lParam), HIWORD(lParam)); break; - case WM_MOVE: - /* Handle the window moving in its own function */ - OnMove(wParam, LOWORD(lParam), HIWORD(lParam)); - break; - case WM_DESTROY: ShowWindow(hDlg, SW_HIDE); TrayIcon_ShellRemoveTrayIcon();

1 0

Vincent Povirk : xmllite: Fix reader_cmp for multiple characters.
by Alexandre Julliard 23 Feb '15

23 Feb '15

Module: wine Branch: master Commit: 8615365945d631df5a3983528963bb2c7a05ff33 URL: http://source.winehq.org/git/wine.git/?a=commit;h=8615365945d631df5a3983528… Author: Vincent Povirk <vincent(a)codeweavers.com> Date: Fri Feb 20 12:54:32 2015 -0600 xmllite: Fix reader_cmp for multiple characters. --- dlls/xmllite/reader.c | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/dlls/xmllite/reader.c b/dlls/xmllite/reader.c index cf89fb7..2aa9d95 100644 --- a/dlls/xmllite/reader.c +++ b/dlls/xmllite/reader.c @@ -900,8 +900,20 @@ static inline WCHAR *reader_get_ptr(xmlreader *reader) static int reader_cmp(xmlreader *reader, const WCHAR *str) { + int i=0; const WCHAR *ptr = reader_get_ptr(reader); - return strncmpW(str, ptr, strlenW(str)); + while (str[i]) + { + if (!ptr[i]) + { + reader_more(reader); + ptr = reader_get_ptr(reader); + } + if (str[i] != ptr[i]) + return ptr[i] - str[i]; + i++; + } + return 0; } /* moves cursor n WCHARs forward */

1 0

← Newer
1
...
12
13
14
15
16
17
18
...
42
Older →

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 February 2015