[PATCH] comdlg32: Explicitly compare to HRESULT values

Nikolay Sivov nsivov at codeweavers.com
Sat Jan 2 05:03:53 CST 2016


Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
---
 dlls/comdlg32/printdlg.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/comdlg32/printdlg.c b/dlls/comdlg32/printdlg.c
index acdb46c..a389e53 100644
--- a/dlls/comdlg32/printdlg.c
+++ b/dlls/comdlg32/printdlg.c
@@ -4126,7 +4126,7 @@ HRESULT WINAPI PrintDlgExA(LPPRINTDLGEXA lppd)
         hr = E_FAIL;
 
     lppd->hDevMode = update_devmode_handleA(lppd->hDevMode, dm);
-    if (!hr && lppd->hDevMode) {
+    if (hr == S_OK && lppd->hDevMode) {
         if (lppd->Flags & PD_RETURNDC) {
             lppd->hDC = CreateDCA(dbuf->pDriverPath, pbuf->pPrinterName, pbuf->pPortName, dm);
             if (!lppd->hDC)
@@ -4274,7 +4274,7 @@ HRESULT WINAPI PrintDlgExW(LPPRINTDLGEXW lppd)
         hr = E_FAIL;
 
     lppd->hDevMode = update_devmode_handleW(lppd->hDevMode, dm);
-    if (!hr && lppd->hDevMode) {
+    if (hr == S_OK && lppd->hDevMode) {
         if (lppd->Flags & PD_RETURNDC) {
             lppd->hDC = CreateDCW(dbuf->pDriverPath, pbuf->pPrinterName, pbuf->pPortName, dm);
             if (!lppd->hDC)
-- 
2.6.4




More information about the wine-patches mailing list