wine-commits August 2012

wine-commits@winehq.org

2 participants
541 discussions

Rico Schüller : d3dx9/tests: Check all elements in the effect parameter value test.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: c178b61bb9ae777d4ffaaba4930cdb179897ee7e URL: http://source.winehq.org/git/wine.git/?a=commit;h=c178b61bb9ae777d4ffaaba49… Author: Rico Schüller <kgbricola(a)web.de> Date: Wed Aug 8 17:02:13 2012 +0200 d3dx9/tests: Check all elements in the effect parameter value test. --- dlls/d3dx9_36/tests/effect.c | 199 ++++++++++++++++++++++------------------- 1 files changed, 107 insertions(+), 92 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=c178b61bb9ae777d4ffaa…

1 0

Rico Schüller : d3dx9/tests: Reduce effect parameter value test count.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 3bd55d0b62905134f1e57136e4ecbc916d034f76 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3bd55d0b62905134f1e57136e… Author: Rico Schüller <kgbricola(a)web.de> Date: Wed Aug 8 17:01:57 2012 +0200 d3dx9/tests: Reduce effect parameter value test count. --- dlls/d3dx9_36/tests/effect.c | 304 ++++++++++++------------------------------ 1 files changed, 85 insertions(+), 219 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=3bd55d0b62905134f1e57…

1 0

Dmitry Timoshkov : include/gdipluspixelformats.h: Avoid explicit TRUE/ FALSE in conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 017f802875ce0a4374eb6bc234a3c3dc0afeb8f4 URL: http://source.winehq.org/git/wine.git/?a=commit;h=017f802875ce0a4374eb6bc23… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Wed Aug 8 17:20:51 2012 +0900 include/gdipluspixelformats.h: Avoid explicit TRUE/FALSE in conditional expressions. --- include/gdipluspixelformats.h | 8 ++++---- 1 files changed, 4 insertions(+), 4 deletions(-) diff --git a/include/gdipluspixelformats.h b/include/gdipluspixelformats.h index 7e6b327..9922b24 100644 --- a/include/gdipluspixelformats.h +++ b/include/gdipluspixelformats.h @@ -51,22 +51,22 @@ typedef INT PixelFormat; static inline BOOL IsIndexedPixelFormat(PixelFormat format) { - return (format & PixelFormatIndexed) ? TRUE : FALSE; + return (format & PixelFormatIndexed) != 0; } static inline BOOL IsAlphaPixelFormat(PixelFormat format) { - return (format & PixelFormatAlpha) ? TRUE : FALSE; + return (format & PixelFormatAlpha) != 0; } static inline BOOL IsCanonicalPixelFormat(PixelFormat format) { - return (format & PixelFormatCanonical) ? TRUE : FALSE; + return (format & PixelFormatCanonical) != 0; } static inline BOOL IsExtendedPixelFormat(PixelFormat format) { - return (format & PixelFormatExtended) ? TRUE : FALSE; + return (format & PixelFormatExtended) != 0; } static inline UINT GetPixelFormatSize(PixelFormat format)

1 0

Michael Stefaniuc : user32: Avoid TRUE:FALSE conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 4affaea0baabba5076f9e4ef7aa202709bf05698 URL: http://source.winehq.org/git/wine.git/?a=commit;h=4affaea0baabba5076f9e4ef7… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Aug 8 10:15:09 2012 +0200 user32: Avoid TRUE:FALSE conditional expressions. --- dlls/user32/combo.c | 4 ++-- dlls/user32/dde_client.c | 6 +++--- dlls/user32/dde_misc.c | 2 +- dlls/user32/edit.c | 2 +- dlls/user32/listbox.c | 4 ++-- dlls/user32/menu.c | 2 +- dlls/user32/sysparams.c | 2 +- dlls/user32/tests/win.c | 2 +- 8 files changed, 12 insertions(+), 12 deletions(-) diff --git a/dlls/user32/combo.c b/dlls/user32/combo.c index 540c64b..d20005c 100644 --- a/dlls/user32/combo.c +++ b/dlls/user32/combo.c @@ -2111,7 +2111,7 @@ LRESULT ComboWndProc_common( HWND hwnd, UINT message, WPARAM wParam, LPARAM lPar if( lParam ) CBGetDroppedControlRect(lphc, (LPRECT)lParam ); return CB_OKAY; case CB_GETDROPPEDSTATE: - return (lphc->wState & CBF_DROPPED) ? TRUE : FALSE; + return (lphc->wState & CBF_DROPPED) != 0; case CB_DIR: return unicode ? SendMessageW(lphc->hWndLBox, LB_DIR, wParam, lParam) : SendMessageA(lphc->hWndLBox, LB_DIR, wParam, lParam); @@ -2173,7 +2173,7 @@ LRESULT ComboWndProc_common( HWND hwnd, UINT message, WPARAM wParam, LPARAM lPar else lphc->wState &= ~CBF_EUI; return CB_OKAY; case CB_GETEXTENDEDUI: - return (lphc->wState & CBF_EUI) ? TRUE : FALSE; + return (lphc->wState & CBF_EUI) != 0; case CB_GETCOMBOBOXINFO: return COMBO_GetComboBoxInfo(lphc, (COMBOBOXINFO *)lParam); case CB_LIMITTEXT: diff --git a/dlls/user32/dde_client.c b/dlls/user32/dde_client.c index b8b8451..ef3ee3f 100644 --- a/dlls/user32/dde_client.c +++ b/dlls/user32/dde_client.c @@ -321,8 +321,8 @@ static WDML_XACT* WDML_ClientQueueAdvise(WDML_CONV* pConv, UINT wType, UINT wFmt /* pack DdeAdvise */ pDdeAdvise = GlobalLock(pXAct->hMem); - pDdeAdvise->fAckReq = (wType & XTYPF_ACKREQ) ? TRUE : FALSE; - pDdeAdvise->fDeferUpd = (wType & XTYPF_NODATA) ? TRUE : FALSE; + pDdeAdvise->fAckReq = (wType & XTYPF_ACKREQ) != 0; + pDdeAdvise->fDeferUpd = (wType & XTYPF_NODATA) != 0; pDdeAdvise->cfFormat = wFmt; GlobalUnlock(pXAct->hMem); @@ -857,7 +857,7 @@ static WDML_QUEUE_STATE WDML_HandleIncomingData(WDML_CONV* pConv, MSG* msg, HDDE * XTYP_ADVDATA and callback should return the proper status. */ pLink = WDML_FindLink(pConv->instance, (HCONV)pConv, WDML_CLIENT_SIDE, hsz, - uiLo ? TRUE : FALSE, wdh.cfFormat); + uiLo != 0, wdh.cfFormat); if (!pLink) { WDML_DecHSZ(pConv->instance, hsz); diff --git a/dlls/user32/dde_misc.c b/dlls/user32/dde_misc.c index 35b9599..724c074 100644 --- a/dlls/user32/dde_misc.c +++ b/dlls/user32/dde_misc.c @@ -2147,7 +2147,7 @@ static BOOL WDML_EnableCallback(WDML_CONV *pConv, UINT wCmd) } if (wCmd == EC_QUERYWAITING) - return pConv->transactions ? TRUE : FALSE; + return pConv->transactions != NULL; if (wCmd != EC_ENABLEALL && wCmd != EC_ENABLEONE) { diff --git a/dlls/user32/edit.c b/dlls/user32/edit.c index ad71a2b..226d737 100644 --- a/dlls/user32/edit.c +++ b/dlls/user32/edit.c @@ -3387,7 +3387,7 @@ static BOOL EDIT_CheckCombo(EDITSTATE *es, UINT msg, INT key) case WM_SYSKEYDOWN: /* Handle Alt+up/down arrows */ if (nEUI) - SendMessageW(hCombo, CB_SHOWDROPDOWN, bDropped ? FALSE : TRUE, 0); + SendMessageW(hCombo, CB_SHOWDROPDOWN, !bDropped, 0); else SendMessageW(hLBox, WM_KEYDOWN, VK_F4, 0); break; diff --git a/dlls/user32/listbox.c b/dlls/user32/listbox.c index 6bf24a7..5686e24 100644 --- a/dlls/user32/listbox.c +++ b/dlls/user32/listbox.c @@ -695,7 +695,7 @@ static void LISTBOX_DrawFocusRect( LB_DESCR *descr, BOOL on ) if (!IsWindowEnabled(descr->self)) SetTextColor( hdc, GetSysColor( COLOR_GRAYTEXT ) ); SetWindowOrgEx( hdc, descr->horz_pos, 0, NULL ); - LISTBOX_PaintItem( descr, hdc, &rect, descr->focus_item, ODA_FOCUS, on ? FALSE : TRUE ); + LISTBOX_PaintItem( descr, hdc, &rect, descr->focus_item, ODA_FOCUS, !on ); if (oldFont) SelectObject( hdc, oldFont ); ReleaseDC( descr->self, hdc ); } @@ -2493,7 +2493,7 @@ static BOOL LISTBOX_Create( HWND hwnd, LPHEADCOMBO lphc ) descr->horz_pos = 0; descr->nb_tabs = 0; descr->tabs = NULL; - descr->caret_on = lphc ? FALSE : TRUE; + descr->caret_on = !lphc; if (descr->style & LBS_NOSEL) descr->caret_on = FALSE; descr->in_focus = FALSE; descr->captured = FALSE; diff --git a/dlls/user32/menu.c b/dlls/user32/menu.c index b113143..f1beb94 100644 --- a/dlls/user32/menu.c +++ b/dlls/user32/menu.c @@ -3125,7 +3125,7 @@ static BOOL MENU_TrackMenu( HMENU hmenu, UINT wFlags, INT x, INT y, /* if the function was called by TrackPopupMenu, continue with the menu tracking. If not, stop it */ else - fEndMenu = ((wFlags & TPM_POPUPMENU) ? FALSE : TRUE); + fEndMenu = !(wFlags & TPM_POPUPMENU); break; diff --git a/dlls/user32/sysparams.c b/dlls/user32/sysparams.c index fecaeda..4603cf4 100644 --- a/dlls/user32/sysparams.c +++ b/dlls/user32/sysparams.c @@ -2063,7 +2063,7 @@ BOOL WINAPI SystemParametersInfoW( UINT uiAction, UINT uiParam, WINE_SPI_FIXME(SPI_SETICONS); /* 88 WINVER >= 0x400 */ case SPI_GETDEFAULTINPUTLANG: /* 89 WINVER >= 0x400 */ - ret = GetKeyboardLayout(0) ? TRUE : FALSE; + ret = GetKeyboardLayout(0) != 0; break; WINE_SPI_FIXME(SPI_SETDEFAULTINPUTLANG); /* 90 WINVER >= 0x400 */ diff --git a/dlls/user32/tests/win.c b/dlls/user32/tests/win.c index 3b7f1a8..cdd0cbc 100644 --- a/dlls/user32/tests/win.c +++ b/dlls/user32/tests/win.c @@ -1172,7 +1172,7 @@ static void test_shell_window(void) ok(!ret, "third call to SetShellWindow(hwnd1)\n"); */ SetWindowLong(hwnd1, GWL_EXSTYLE, GetWindowLong(hwnd1,GWL_EXSTYLE)|WS_EX_TOPMOST); - ret = GetWindowLong(hwnd1,GWL_EXSTYLE)&WS_EX_TOPMOST? TRUE: FALSE; + ret = (GetWindowLong(hwnd1,GWL_EXSTYLE) & WS_EX_TOPMOST) != 0; ok(!ret, "SetWindowExStyle(hwnd1, WS_EX_TOPMOST)\n"); ret = DestroyWindow(hwnd1);

1 0

Michael Stefaniuc : ole32: Avoid TRUE:FALSE conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 43ae84636aa7084cf10fbab9957e0342bc17be03 URL: http://source.winehq.org/git/wine.git/?a=commit;h=43ae84636aa7084cf10fbab99… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Aug 8 10:13:54 2012 +0200 ole32: Avoid TRUE:FALSE conditional expressions. --- dlls/ole32/compobj.c | 5 ++--- dlls/ole32/defaulthandler.c | 2 +- dlls/ole32/ole2.c | 4 ++-- dlls/ole32/tests/clipboard.c | 2 +- dlls/ole32/tests/compobj.c | 2 +- 5 files changed, 7 insertions(+), 8 deletions(-) diff --git a/dlls/ole32/compobj.c b/dlls/ole32/compobj.c index ff8f961..4f6229d 100644 --- a/dlls/ole32/compobj.c +++ b/dlls/ole32/compobj.c @@ -3769,10 +3769,9 @@ HRESULT WINAPI CoWaitForMultipleHandles(DWORD dwFlags, DWORD dwTimeout, { TRACE("waiting for rpc completion\n"); - res = WaitForMultipleObjectsEx(cHandles, pHandles, - (dwFlags & COWAIT_WAITALL) ? TRUE : FALSE, + res = WaitForMultipleObjectsEx(cHandles, pHandles, (dwFlags & COWAIT_WAITALL) != 0, (dwTimeout == INFINITE) ? INFINITE : start_time + dwTimeout - now, - (dwFlags & COWAIT_ALERTABLE) ? TRUE : FALSE); + (dwFlags & COWAIT_ALERTABLE) != 0); } switch (res) diff --git a/dlls/ole32/defaulthandler.c b/dlls/ole32/defaulthandler.c index b43b962..a2a30de 100644 --- a/dlls/ole32/defaulthandler.c +++ b/dlls/ole32/defaulthandler.c @@ -1899,7 +1899,7 @@ static DefaultHandler* DefaultHandler_Construct( This->IAdviseSink_iface.lpVtbl = &DefaultHandler_IAdviseSink_VTable; This->IPersistStorage_iface.lpVtbl = &DefaultHandler_IPersistStorage_VTable; - This->inproc_server = (flags & EMBDHLP_INPROC_SERVER) ? TRUE : FALSE; + This->inproc_server = (flags & EMBDHLP_INPROC_SERVER) != 0; /* * Start with one reference count. The caller of this function diff --git a/dlls/ole32/ole2.c b/dlls/ole32/ole2.c index 6d37aa6..74fb43e 100644 --- a/dlls/ole32/ole2.c +++ b/dlls/ole32/ole2.c @@ -290,7 +290,7 @@ static inline HANDLE get_droptarget_handle(HWND hwnd) */ static inline BOOL is_droptarget(HWND hwnd) { - return get_droptarget_handle(hwnd) ? TRUE : FALSE; + return get_droptarget_handle(hwnd) != 0; } /************************************************************* @@ -1660,7 +1660,7 @@ static BOOL OLEMenu_SetIsServerMenu( HMENU hmenu, OleMenuDescriptor *pOleMenuDes if ( nPos < nWidth ) { /* Odd elements are server menu widths */ - pOleMenuDescriptor->bIsServerItem = (i%2) ? TRUE : FALSE; + pOleMenuDescriptor->bIsServerItem = i%2; break; } } diff --git a/dlls/ole32/tests/clipboard.c b/dlls/ole32/tests/clipboard.c index 1029a68..5a16032 100644 --- a/dlls/ole32/tests/clipboard.c +++ b/dlls/ole32/tests/clipboard.c @@ -740,7 +740,7 @@ static void test_cf_dataobject(IDataObject *data) break; } cfs_seen[count] = fmt.cfFormat; - ok(fmt_ptr->first_use_of_cf == seen_cf ? FALSE : TRUE, "got %08x expected %08x\n", + ok(fmt_ptr->first_use_of_cf != seen_cf, "got %08x expected %08x\n", fmt_ptr->first_use_of_cf, !seen_cf); ok(fmt_ptr->res[0] == 0, "got %08x\n", fmt_ptr->res[1]); ok(fmt_ptr->res[1] == 0, "got %08x\n", fmt_ptr->res[2]); diff --git a/dlls/ole32/tests/compobj.c b/dlls/ole32/tests/compobj.c index b76c9d2..9dfc271 100644 --- a/dlls/ole32/tests/compobj.c +++ b/dlls/ole32/tests/compobj.c @@ -1110,7 +1110,7 @@ static DWORD CALLBACK free_libraries_thread(LPVOID p) static inline BOOL is_module_loaded(const char *module) { - return GetModuleHandle(module) ? TRUE : FALSE; + return GetModuleHandle(module) != 0; } static void test_CoFreeUnusedLibraries(void)

1 0

Michael Stefaniuc : server: Avoid TRUE:FALSE conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 794ad909825763984979ef1d8a9f035c45d78a0d URL: http://source.winehq.org/git/wine.git/?a=commit;h=794ad909825763984979ef1d8… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Aug 8 10:10:44 2012 +0200 server: Avoid TRUE:FALSE conditional expressions. --- server/security.h | 4 ++-- server/token.c | 6 +++--- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/server/security.h b/server/security.h index d5f629d..38e36c0 100644 --- a/server/security.h +++ b/server/security.h @@ -90,7 +90,7 @@ extern int sd_is_valid( const struct security_descriptor *sd, data_size_t size ) /* gets the discretionary access control list from a security descriptor */ static inline const ACL *sd_get_dacl( const struct security_descriptor *sd, int *present ) { - *present = (sd->control & SE_DACL_PRESENT ? TRUE : FALSE); + *present = (sd->control & SE_DACL_PRESENT) != 0; if (sd->dacl_len) return (const ACL *)((const char *)(sd + 1) + @@ -102,7 +102,7 @@ static inline const ACL *sd_get_dacl( const struct security_descriptor *sd, int /* gets the system access control list from a security descriptor */ static inline const ACL *sd_get_sacl( const struct security_descriptor *sd, int *present ) { - *present = (sd->control & SE_SACL_PRESENT ? TRUE : FALSE); + *present = (sd->control & SE_SACL_PRESENT) != 0; if (sd->sacl_len) return (const ACL *)((const char *)(sd + 1) + diff --git a/server/token.c b/server/token.c index a0ec143..b66a603 100644 --- a/server/token.c +++ b/server/token.c @@ -475,9 +475,9 @@ static struct token *create_token( unsigned primary, const SID *user, memcpy( &group->sid, groups[i].Sid, FIELD_OFFSET( SID, SubAuthority[((const SID *)groups[i].Sid)->SubAuthorityCount] ) ); group->enabled = TRUE; group->def = TRUE; - group->logon = (groups[i].Attributes & SE_GROUP_LOGON_ID) ? TRUE : FALSE; - group->mandatory = (groups[i].Attributes & SE_GROUP_MANDATORY) ? TRUE : FALSE; - group->owner = groups[i].Attributes & SE_GROUP_OWNER ? TRUE : FALSE; + group->logon = (groups[i].Attributes & SE_GROUP_LOGON_ID) != 0; + group->mandatory = (groups[i].Attributes & SE_GROUP_MANDATORY) != 0; + group->owner = (groups[i].Attributes & SE_GROUP_OWNER) != 0; group->resource = FALSE; group->deny_only = FALSE; list_add_tail( &token->groups, &group->entry );

1 0

Michael Stefaniuc : krnl386.exe16: Avoid TRUE:FALSE conditional expressions .
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: dfa57c6e97f1cbbd8147ee7986dc39755ef51c0d URL: http://source.winehq.org/git/wine.git/?a=commit;h=dfa57c6e97f1cbbd8147ee798… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 6 11:26:07 2012 +0200 krnl386.exe16: Avoid TRUE:FALSE conditional expressions. --- dlls/krnl386.exe16/dosexe.c | 2 +- dlls/krnl386.exe16/int31.c | 2 +- dlls/krnl386.exe16/vga.c | 2 +- 3 files changed, 3 insertions(+), 3 deletions(-) diff --git a/dlls/krnl386.exe16/dosexe.c b/dlls/krnl386.exe16/dosexe.c index ff55326..9797c86 100644 --- a/dlls/krnl386.exe16/dosexe.c +++ b/dlls/krnl386.exe16/dosexe.c @@ -61,7 +61,7 @@ static BOOL DOSVM_isdosexe; */ BOOL DOSVM_IsWin16(void) { - return DOSVM_isdosexe ? FALSE : TRUE; + return !DOSVM_isdosexe; } /********************************************************************** diff --git a/dlls/krnl386.exe16/int31.c b/dlls/krnl386.exe16/int31.c index 9828876..ac9ae1f 100644 --- a/dlls/krnl386.exe16/int31.c +++ b/dlls/krnl386.exe16/int31.c @@ -93,7 +93,7 @@ typedef struct { */ BOOL DOSVM_IsDos32(void) { - return (dpmi_flag & 1) ? TRUE : FALSE; + return (dpmi_flag & 1) != 0; } diff --git a/dlls/krnl386.exe16/vga.c b/dlls/krnl386.exe16/vga.c index 70b3ef1..98edcef 100644 --- a/dlls/krnl386.exe16/vga.c +++ b/dlls/krnl386.exe16/vga.c @@ -716,7 +716,7 @@ static void VGA_InstallTimer(unsigned Rate) static BOOL VGA_IsTimerRunning(void) { - return VGA_timer_thread ? TRUE : FALSE; + return VGA_timer_thread != 0; } static HANDLE VGA_AlphaConsole(void)

1 0

Michael Stefaniuc : crypt32: Avoid TRUE:FALSE conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 2793be9c6ddd2bf6331335a959fcda2408aa74d3 URL: http://source.winehq.org/git/wine.git/?a=commit;h=2793be9c6ddd2bf6331335a95… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 6 11:19:22 2012 +0200 crypt32: Avoid TRUE:FALSE conditional expressions. --- dlls/crypt32/base64.c | 4 ++-- dlls/crypt32/decode.c | 2 +- dlls/crypt32/message.c | 9 ++++----- dlls/crypt32/oid.c | 6 +++--- dlls/crypt32/store.c | 6 +++--- 5 files changed, 13 insertions(+), 14 deletions(-) diff --git a/dlls/crypt32/base64.c b/dlls/crypt32/base64.c index a01a7b6..9d6dcb4 100644 --- a/dlls/crypt32/base64.c +++ b/dlls/crypt32/base64.c @@ -827,7 +827,7 @@ BOOL WINAPI CryptStringToBinaryA(LPCSTR pszString, ret = decoder(pszString, cchString, pbBinary, pcbBinary, pdwSkip, pdwFlags); if (ret) SetLastError(ret); - return (ret == ERROR_SUCCESS) ? TRUE : FALSE; + return ret == ERROR_SUCCESS; } static LONG decodeBase64BlockW(const WCHAR *in_buf, int in_len, @@ -1159,5 +1159,5 @@ BOOL WINAPI CryptStringToBinaryW(LPCWSTR pszString, ret = decoder(pszString, cchString, pbBinary, pcbBinary, pdwSkip, pdwFlags); if (ret) SetLastError(ret); - return (ret == ERROR_SUCCESS) ? TRUE : FALSE; + return ret == ERROR_SUCCESS; } diff --git a/dlls/crypt32/decode.c b/dlls/crypt32/decode.c index bdcb744..bfd61d0 100644 --- a/dlls/crypt32/decode.c +++ b/dlls/crypt32/decode.c @@ -2977,7 +2977,7 @@ static BOOL CRYPT_AsnDecodeBool(const BYTE *pbEncoded, DWORD cbEncoded, else { *pcbStructInfo = sizeof(BOOL); - *(BOOL *)pvStructInfo = pbEncoded[2] ? TRUE : FALSE; + *(BOOL *)pvStructInfo = pbEncoded[2] != 0; ret = TRUE; } TRACE("returning %d (%08x)\n", ret, GetLastError()); diff --git a/dlls/crypt32/message.c b/dlls/crypt32/message.c index e5b6f2a..8668cb1 100644 --- a/dlls/crypt32/message.c +++ b/dlls/crypt32/message.c @@ -140,7 +140,7 @@ BOOL WINAPI CryptVerifyDetachedMessageSignature( for (i = 0; ret && i < cToBeSigned; i++) ret = CryptMsgUpdate(msg, rgpbToBeSigned[i], rgcbToBeSigned[i], - i == cToBeSigned - 1 ? TRUE : FALSE); + i == cToBeSigned - 1); } if (ret) { @@ -304,8 +304,7 @@ BOOL WINAPI CryptHashMessage(PCRYPT_HASH_MESSAGE_PARA pHashPara, if (msg) { for (i = 0, ret = TRUE; ret && i < cToBeHashed; i++) - ret = CryptMsgUpdate(msg, rgpbToBeHashed[i], rgcbToBeHashed[i], - i == cToBeHashed - 1 ? TRUE : FALSE); + ret = CryptMsgUpdate(msg, rgpbToBeHashed[i], rgcbToBeHashed[i], i == cToBeHashed - 1); if (ret) { ret = CryptMsgGetParam(msg, CMSG_CONTENT_PARAM, 0, pbHashedBlob, @@ -356,7 +355,7 @@ BOOL WINAPI CryptVerifyDetachedMessageHash(PCRYPT_HASH_MESSAGE_PARA pHashPara, for (i = 0; ret && i < cToBeHashed; i++) { ret = CryptMsgUpdate(msg, rgpbToBeHashed[i], - rgcbToBeHashed[i], i == cToBeHashed - 1 ? TRUE : FALSE); + rgcbToBeHashed[i], i == cToBeHashed - 1); } } else @@ -511,7 +510,7 @@ BOOL WINAPI CryptSignMessage(PCRYPT_SIGN_MESSAGE_PARA pSignPara, for (i = 0; ret && i < cToBeSigned; ++i) { ret = CryptMsgUpdate(msg, rgpbToBeSigned[i], rgcbToBeSigned[i], - i == cToBeSigned - 1 ? TRUE : FALSE); + i == cToBeSigned - 1); } } else diff --git a/dlls/crypt32/oid.c b/dlls/crypt32/oid.c index d56fe83..4f09e7b 100644 --- a/dlls/crypt32/oid.c +++ b/dlls/crypt32/oid.c @@ -684,7 +684,7 @@ BOOL WINAPI CryptUnregisterOIDFunction(DWORD dwEncodingType, LPCSTR pszFuncName, CryptMemFree(szKey); if (rc) SetLastError(rc); - return rc ? FALSE : TRUE; + return !rc; } BOOL WINAPI CryptGetOIDFunctionValue(DWORD dwEncodingType, LPCSTR pszFuncName, @@ -721,7 +721,7 @@ BOOL WINAPI CryptGetOIDFunctionValue(DWORD dwEncodingType, LPCSTR pszFuncName, SetLastError(rc); RegCloseKey(hKey); } - return rc ? FALSE : TRUE; + return !rc; } BOOL WINAPI CryptSetOIDFunctionValue(DWORD dwEncodingType, LPCSTR pszFuncName, @@ -758,7 +758,7 @@ BOOL WINAPI CryptSetOIDFunctionValue(DWORD dwEncodingType, LPCSTR pszFuncName, SetLastError(rc); RegCloseKey(hKey); } - return rc ? FALSE : TRUE; + return !rc; } static LPCWSTR CRYPT_FindStringInMultiString(LPCWSTR multi, LPCWSTR toFind) diff --git a/dlls/crypt32/store.c b/dlls/crypt32/store.c index b540e61..6d27fc1 100644 --- a/dlls/crypt32/store.c +++ b/dlls/crypt32/store.c @@ -163,7 +163,7 @@ static BOOL CRYPT_MemAddCert(PWINECRYPT_CERTSTORE store, void *cert, if (ppStoreContext) *ppStoreContext = CertDuplicateCertificateContext(context); } - return context ? TRUE : FALSE; + return context != 0; } static void *CRYPT_MemEnumCert(PWINECRYPT_CERTSTORE store, void *pPrev) @@ -208,7 +208,7 @@ static BOOL CRYPT_MemAddCrl(PWINECRYPT_CERTSTORE store, void *crl, if (ppStoreContext) *ppStoreContext = CertDuplicateCRLContext(context); } - return context ? TRUE : FALSE; + return context != 0; } static void *CRYPT_MemEnumCrl(PWINECRYPT_CERTSTORE store, void *pPrev) @@ -253,7 +253,7 @@ static BOOL CRYPT_MemAddCtl(PWINECRYPT_CERTSTORE store, void *ctl, if (ppStoreContext) *ppStoreContext = CertDuplicateCTLContext(context); } - return context ? TRUE : FALSE; + return context != 0; } static void *CRYPT_MemEnumCtl(PWINECRYPT_CERTSTORE store, void *pPrev)

1 0

Michael Stefaniuc : avifil32: Avoid TRUE:FALSE conditional expressions.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: fdfb86589e19e2c433522a20615a071efb72a048 URL: http://source.winehq.org/git/wine.git/?a=commit;h=fdfb86589e19e2c433522a206… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Mon Aug 6 10:56:28 2012 +0200 avifil32: Avoid TRUE:FALSE conditional expressions. --- dlls/avifil32/api.c | 2 +- dlls/avifil32/getframe.c | 3 +-- 2 files changed, 2 insertions(+), 3 deletions(-) diff --git a/dlls/avifil32/api.c b/dlls/avifil32/api.c index c53897a..6c07a2f 100644 --- a/dlls/avifil32/api.c +++ b/dlls/avifil32/api.c @@ -1244,7 +1244,7 @@ static BOOL AVISaveOptionsFmtChoose(HWND hWnd) pOptions->dwFlags |= AVICOMPRESSF_VALID; HeapFree(GetProcessHeap(), 0, afmtc.pwfxEnum); - return (ret == S_OK ? TRUE : FALSE); + return ret == S_OK; } else { ERR(": unknown streamtype 0x%08X\n", sInfo.fccType); return FALSE; diff --git a/dlls/avifil32/getframe.c b/dlls/avifil32/getframe.c index c5b2bd4..46b1683 100644 --- a/dlls/avifil32/getframe.c +++ b/dlls/avifil32/getframe.c @@ -307,8 +307,7 @@ static HRESULT WINAPI IGetFrame_fnSetFormat(IGetFrame *iface, if (sInfo.fccType != streamtypeVIDEO) return AVIERR_UNSUPPORTED; - This->bFormatChanges = - (sInfo.dwFlags & AVISTREAMINFO_FORMATCHANGES ? TRUE : FALSE ); + This->bFormatChanges = (sInfo.dwFlags & AVISTREAMINFO_FORMATCHANGES) != 0; This->dwFormatChangeCount = sInfo.dwFormatChangeCount; This->dwEditCount = sInfo.dwEditCount; This->lCurrentFrame = -1;

1 0

Dmitry Timoshkov : gdiplus: Correct GdipSaveImageToFile spec entry.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: af042f2754682fd7d819ee1bc6a3688048dea79b URL: http://source.winehq.org/git/wine.git/?a=commit;h=af042f2754682fd7d819ee1bc… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Aug 6 11:57:26 2012 +0900 gdiplus: Correct GdipSaveImageToFile spec entry. --- dlls/gdiplus/gdiplus.spec | 2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/dlls/gdiplus/gdiplus.spec b/dlls/gdiplus/gdiplus.spec index 7d6c51f..ccf47f9 100644 --- a/dlls/gdiplus/gdiplus.spec +++ b/dlls/gdiplus/gdiplus.spec @@ -474,7 +474,7 @@ 474 stdcall GdipSaveAdd(ptr ptr) 475 stub GdipSaveAddImage 476 stdcall GdipSaveGraphics(ptr ptr) -477 stdcall GdipSaveImageToFile(ptr ptr ptr ptr) +477 stdcall GdipSaveImageToFile(ptr wstr ptr ptr) 478 stdcall GdipSaveImageToStream(ptr ptr ptr ptr) 479 stdcall GdipScaleLineTransform(ptr float float long) 480 stdcall GdipScaleMatrix(ptr float float long)

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 August 2012