August 2012 - Wine-commits - list.winehq.org

Eric Pouech : msacm32: Convert acmFormatChoose internals to Unicode, implement acmFormatChoose[AW] on top of it.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 22789aa30ef94fed0f74f8623e1088716ec3ce14 URL: http://source.winehq.org/git/wine.git/?a=commit;h=22789aa30ef94fed0f74f8623… Author: Eric Pouech <eric.pouech(a)orange.fr> Date: Sun Aug 12 10:03:41 2012 +0200 msacm32: Convert acmFormatChoose internals to Unicode, implement acmFormatChoose[AW] on top of it. --- dlls/msacm32/format.c | 181 ++++++++++++++++++++++++++++++++++--------------- 1 files changed, 125 insertions(+), 56 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=22789aa30ef94fed0f74f…

1 0

Lauri Kenttä : po: Update Finnish translation.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 8399033b390a9c65ac7fff246c1fc56ce8cb802e URL: http://source.winehq.org/git/wine.git/?a=commit;h=8399033b390a9c65ac7fff246… Author: Lauri Kenttä <lauri.kentta(a)gmail.com> Date: Sun Aug 5 20:16:35 2012 +0300 po: Update Finnish translation. --- po/fi.po | 6 +++--- 1 files changed, 3 insertions(+), 3 deletions(-) diff --git a/po/fi.po b/po/fi.po index d53d735..26d221f 100644 --- a/po/fi.po +++ b/po/fi.po @@ -3450,16 +3450,16 @@ msgid "Test Force Feedback" msgstr "Testaa voimapalautetta" #: joy.rc:63 -#, fuzzy -#| msgid "Available formats" msgid "Available Effects" -msgstr "Mahdolliset muodot" +msgstr "Mahdolliset efektit" #: joy.rc:65 msgid "" "Press any button in the controller to activate the chosen effect. The effect " "direction can be changed with the controller axis." msgstr "" +"Aktivoi valittu efekti painamalla mitä tahansa ohjaimen painiketta. Efektin " +"suuntaa voi muuttaa ohjaimen akselilla." #: joy.rc:28 msgid "Game Controllers"

1 0

Nikolay Sivov : riched20: Use proper interface methods macros.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: e3f0d4084006be0473431f9bc498c6f526b4a555 URL: http://source.winehq.org/git/wine.git/?a=commit;h=e3f0d4084006be0473431f9bc… Author: Nikolay Sivov <nsivov(a)codeweavers.com> Date: Wed Aug 1 14:56:19 2012 +0400 riched20: Use proper interface methods macros. --- dlls/riched20/editor.c | 10 +++++----- 1 files changed, 5 insertions(+), 5 deletions(-) diff --git a/dlls/riched20/editor.c b/dlls/riched20/editor.c index 44c7ad2..6e0fb3c 100644 --- a/dlls/riched20/editor.c +++ b/dlls/riched20/editor.c @@ -2793,8 +2793,8 @@ static void ME_DestroyEditor(ME_TextEditor *editor) if (editor->rgbBackColor != -1) DeleteObject(editor->hbrBackground); if(editor->lpOleCallback) - IUnknown_Release(editor->lpOleCallback); - IUnknown_Release(editor->texthost); + IRichEditOleCallback_Release(editor->lpOleCallback); + ITextHost_Release(editor->texthost); OleUninitialize(); FREE_OBJ(editor->pBuffer); @@ -4311,10 +4311,10 @@ LRESULT ME_HandleMessage(ME_TextEditor *editor, UINT msg, WPARAM wParam, } case EM_SETOLECALLBACK: if(editor->lpOleCallback) - IUnknown_Release(editor->lpOleCallback); - editor->lpOleCallback = (LPRICHEDITOLECALLBACK)lParam; + IRichEditOleCallback_Release(editor->lpOleCallback); + editor->lpOleCallback = (IRichEditOleCallback*)lParam; if(editor->lpOleCallback) - IUnknown_AddRef(editor->lpOleCallback); + IRichEditOleCallback_AddRef(editor->lpOleCallback); return TRUE; case EM_GETWORDBREAKPROC: return (LRESULT)editor->pfnWordBreak;

1 0

Dmitry Timoshkov : windowscodecs: Handle short reads in the metadata handler.
by Alexandre Julliard 13 Aug '12

13 Aug '12

Module: wine Branch: master Commit: 5f4b8f5916a681c9a8206ba649557177c3e788ca URL: http://source.winehq.org/git/wine.git/?a=commit;h=5f4b8f5916a681c9a8206ba64… Author: Dmitry Timoshkov <dmitry(a)baikal.ru> Date: Mon Aug 6 11:07:49 2012 +0900 windowscodecs: Handle short reads in the metadata handler. --- dlls/windowscodecs/metadatahandler.c | 63 ++++++++++++++++++++------------- 1 files changed, 38 insertions(+), 25 deletions(-) diff --git a/dlls/windowscodecs/metadatahandler.c b/dlls/windowscodecs/metadatahandler.c index 042d6ae..084facf 100644 --- a/dlls/windowscodecs/metadatahandler.c +++ b/dlls/windowscodecs/metadatahandler.c @@ -681,7 +681,8 @@ static HRESULT LoadUnknownMetadata(IStream *input, const GUID *preferred_vendor, if (!data) return E_OUTOFMEMORY; hr = IStream_Read(input, data, stat.cbSize.QuadPart, &bytesread); - if (FAILED(hr)) + if (bytesread != stat.cbSize.QuadPart) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, data); return hr; @@ -770,7 +771,7 @@ static int tag_to_vt(SHORT tag) static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, MetadataItem *item, BOOL native_byte_order) { - ULONG count, value, i; + ULONG count, value, i, bytesread; SHORT type; LARGE_INTEGER pos; HRESULT hr; @@ -822,8 +823,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.caub.pElems); return hr; } - hr = IStream_Read(input, item->value.u.caub.pElems, count, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.caub.pElems, count, &bytesread); + if (bytesread != count) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.caub.pElems); return hr; @@ -866,8 +868,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.caui.pElems); return hr; } - hr = IStream_Read(input, item->value.u.caui.pElems, count * 2, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.caui.pElems, count * 2, &bytesread); + if (bytesread != count * 2) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.caui.pElems); return hr; @@ -898,8 +901,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.caul.pElems); return hr; } - hr = IStream_Read(input, item->value.u.caul.pElems, count * 4, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.caul.pElems, count * 4, &bytesread); + if (bytesread != count * 4) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.caul.pElems); return hr; @@ -925,8 +929,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, hr = IStream_Seek(input, pos, SEEK_SET, NULL); if (FAILED(hr)) return hr; - hr = IStream_Read(input, &ull, sizeof(ull), NULL); - if (FAILED(hr)) return hr; + hr = IStream_Read(input, &ull, sizeof(ull), &bytesread); + if (bytesread != sizeof(ull)) hr = E_FAIL; + if (hr != S_OK) return hr; item->value.u.uhVal.QuadPart = ull; @@ -953,8 +958,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.cauh.pElems); return hr; } - hr = IStream_Read(input, item->value.u.cauh.pElems, count * 8, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.cauh.pElems, count * 8, &bytesread); + if (bytesread != count * 8) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.cauh.pElems); return hr; @@ -990,8 +996,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.pszVal); return hr; } - hr = IStream_Read(input, item->value.u.pszVal, count, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.pszVal, count, &bytesread); + if (bytesread != count) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.pszVal); return hr; @@ -1025,8 +1032,9 @@ static HRESULT load_IFD_entry(IStream *input, const struct IFD_entry *entry, HeapFree(GetProcessHeap(), 0, item->value.u.blob.pBlobData); return hr; } - hr = IStream_Read(input, item->value.u.blob.pBlobData, count, NULL); - if (FAILED(hr)) + hr = IStream_Read(input, item->value.u.blob.pBlobData, count, &bytesread); + if (bytesread != count) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, item->value.u.blob.pBlobData); return hr; @@ -1047,6 +1055,7 @@ static HRESULT LoadIfdMetadata(IStream *input, const GUID *preferred_vendor, USHORT count, i; struct IFD_entry *entry; BOOL native_byte_order = TRUE; + ULONG bytesread; TRACE("\n"); @@ -1057,16 +1066,18 @@ static HRESULT LoadIfdMetadata(IStream *input, const GUID *preferred_vendor, #endif native_byte_order = FALSE; - hr = IStream_Read(input, &count, sizeof(count), NULL); - if (FAILED(hr)) return hr; + hr = IStream_Read(input, &count, sizeof(count), &bytesread); + if (bytesread != sizeof(count)) hr = E_FAIL; + if (hr != S_OK) return hr; SWAP_USHORT(count); entry = HeapAlloc(GetProcessHeap(), 0, count * sizeof(*entry)); if (!entry) return E_OUTOFMEMORY; - hr = IStream_Read(input, entry, count * sizeof(*entry), NULL); - if (FAILED(hr)) + hr = IStream_Read(input, entry, count * sizeof(*entry), &bytesread); + if (bytesread != count * sizeof(*entry)) hr = E_FAIL; + if (hr != S_OK) { HeapFree(GetProcessHeap(), 0, entry); return hr; @@ -1079,8 +1090,9 @@ static HRESULT LoadIfdMetadata(IStream *input, const GUID *preferred_vendor, LARGE_INTEGER pos; USHORT next_ifd_count; - hr = IStream_Read(input, &next_ifd_offset, sizeof(next_ifd_offset), NULL); - if (FAILED(hr)) break; + hr = IStream_Read(input, &next_ifd_offset, sizeof(next_ifd_offset), &bytesread); + if (bytesread != sizeof(next_ifd_offset)) hr = E_FAIL; + if (hr != S_OK) break; SWAP_ULONG(next_ifd_offset); if (!next_ifd_offset) break; @@ -1089,8 +1101,9 @@ static HRESULT LoadIfdMetadata(IStream *input, const GUID *preferred_vendor, hr = IStream_Seek(input, pos, SEEK_SET, NULL); if (FAILED(hr)) break; - hr = IStream_Read(input, &next_ifd_count, sizeof(next_ifd_count), NULL); - if (FAILED(hr)) break; + hr = IStream_Read(input, &next_ifd_count, sizeof(next_ifd_count), &bytesread); + if (bytesread != sizeof(next_ifd_count)) hr = E_FAIL; + if (hr != S_OK) break; SWAP_USHORT(next_ifd_count); @@ -1099,7 +1112,7 @@ static HRESULT LoadIfdMetadata(IStream *input, const GUID *preferred_vendor, if (FAILED(hr)) break; } - if (FAILED(hr) || i == 4096) + if (hr != S_OK || i == 4096) { HeapFree(GetProcessHeap(), 0, entry); return WINCODEC_ERR_BADMETADATAHEADER;

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

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

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 : 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 : 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 : 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