Detlef Riekenberg : advpack+setupapi: Remove unneeded checks before SetupCloseInfFile.

Alexandre Julliard julliard at wine.codeweavers.com
Tue Jul 10 08:02:33 CDT 2007


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

Author: Detlef Riekenberg <wine.dev at web.de>
Date:   Mon Jul  9 19:38:59 2007 +0200

advpack+setupapi: Remove unneeded checks before SetupCloseInfFile.

---

 dlls/advpack/install.c       |    3 +--
 dlls/setupapi/tests/parser.c |    2 +-
 2 files changed, 2 insertions(+), 3 deletions(-)

diff --git a/dlls/advpack/install.c b/dlls/advpack/install.c
index b7e997c..c5a5df0 100644
--- a/dlls/advpack/install.c
+++ b/dlls/advpack/install.c
@@ -512,8 +512,7 @@ static HRESULT install_init(LPCWSTR inf_filename, LPCWSTR install_sec,
 /* release the install instance information */
 static void install_release(const ADVInfo *info)
 {
-    if (info->hinf && info->hinf != INVALID_HANDLE_VALUE)
-        SetupCloseInfFile(info->hinf);
+    SetupCloseInfFile(info->hinf);
 
     HeapFree(GetProcessHeap(), 0, info->inf_path);
     HeapFree(GetProcessHeap(), 0, info->inf_filename);
diff --git a/dlls/setupapi/tests/parser.c b/dlls/setupapi/tests/parser.c
index 2585f6d..2aaaa5e 100644
--- a/dlls/setupapi/tests/parser.c
+++ b/dlls/setupapi/tests/parser.c
@@ -167,7 +167,7 @@ static void test_invalid_files(void)
             ok( hinf != INVALID_HANDLE_VALUE, "file %u: Open failed\n", i );
             ok( err == 0, "file %u: Error code set to %u\n", i, err );
         }
-        if (hinf != INVALID_HANDLE_VALUE) SetupCloseInfFile( hinf );
+        SetupCloseInfFile( hinf );
     }
 }
 




More information about the wine-cvs mailing list