Nikolay Sivov : comdlg32: Explicitly compare to HRESULT values.

Alexandre Julliard julliard at wine.codeweavers.com
Sun Jan 3 14:24:16 CST 2016


Module: wine
Branch: master
Commit: c8e407da3badb4d5bc7b6deba0e8a093f8245179
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=c8e407da3badb4d5bc7b6deba0e8a093f8245179

Author: Nikolay Sivov <nsivov at codeweavers.com>
Date:   Sat Jan  2 14:03:53 2016 +0300

comdlg32: Explicitly compare to HRESULT values.

Signed-off-by: Nikolay Sivov <nsivov at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 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)




More information about the wine-cvs mailing list