Hugh McMaster : regsvr32: Move silent check to the output_write function.

[Alexandre Julliard]

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

Author: Hugh McMaster <hugh.mcmaster at masterindexing.com>
Date:   Wed Mar 19 12:03:49 2014 +1100

regsvr32: Move silent check to the output_write function.

---

 programs/regsvr32/regsvr32.c |   50 ++++++++++++++----------------------------
 1 file changed, 17 insertions(+), 33 deletions(-)

diff --git a/programs/regsvr32/regsvr32.c b/programs/regsvr32/regsvr32.c
index c795f42..624acd6 100644
--- a/programs/regsvr32/regsvr32.c
+++ b/programs/regsvr32/regsvr32.c
@@ -72,6 +72,8 @@ static void __cdecl output_write(UINT id, ...)
     char *str;
     DWORD len, nOut, ret;
 
+    if (Silent) return;
+
     if (!LoadStringA(GetModuleHandleA(NULL), id, fmt, sizeof(fmt)/sizeof(fmt[0])))
     {
         WINE_FIXME("LoadString failed with %d\n", GetLastError());
@@ -112,16 +114,13 @@ static VOID *LoadProc(const char* strDll, const char* procName, HMODULE* DllHand
     *DllHandle = LoadLibraryExA(strDll, 0, LOAD_WITH_ALTERED_SEARCH_PATH);
     if(!*DllHandle)
     {
-        if(!Silent)
-            output_write(STRING_DLL_LOAD_FAILED, strDll);
-
+        output_write(STRING_DLL_LOAD_FAILED, strDll);
         ExitProcess(1);
     }
     proc = (VOID *) GetProcAddress(*DllHandle, procName);
     if(!proc)
     {
-        if(!Silent)
-            output_write(STRING_PROC_NOT_IMPLEMENTED, procName, strDll);
+        output_write(STRING_PROC_NOT_IMPLEMENTED, procName, strDll);
         FreeLibrary(*DllHandle);
         return NULL;
     }
@@ -141,13 +140,10 @@ static int RegisterDll(const char* strDll)
     hr = pfRegister();
     if(FAILED(hr))
     {
-        if(!Silent)
-            output_write(STRING_REGISTER_FAILED, strDll);
-
+        output_write(STRING_REGISTER_FAILED, strDll);
         return -1;
     }
-    if(!Silent)
-        output_write(STRING_REGISTER_SUCCESSFUL, strDll);
+    output_write(STRING_REGISTER_SUCCESSFUL, strDll);
 
     if(DllHandle)
         FreeLibrary(DllHandle);
@@ -167,13 +163,10 @@ static int UnregisterDll(char* strDll)
     hr = pfUnregister();
     if(FAILED(hr))
     {
-        if(!Silent)
-            output_write(STRING_UNREGISTER_FAILED, strDll);
-
+        output_write(STRING_UNREGISTER_FAILED, strDll);
         return -1;
     }
-    if(!Silent)
-        output_write(STRING_UNREGISTER_SUCCESSFUL, strDll);
+    output_write(STRING_UNREGISTER_SUCCESSFUL, strDll);
 
     if(DllHandle)
         FreeLibrary(DllHandle);
@@ -193,22 +186,16 @@ static int InstallDll(BOOL install, char *strDll, WCHAR *command_line)
     hr = pfInstall(install, command_line);
     if(FAILED(hr))
     {
-        if(!Silent)
-        {
-            if (install)
-                output_write(STRING_INSTALL_FAILED, strDll);
-            else
-                output_write(STRING_UNINSTALL_FAILED, strDll);
-        }
-        return -1;
-    }
-    if(!Silent)
-    {
         if (install)
-            output_write(STRING_INSTALL_SUCCESSFUL, strDll);
+            output_write(STRING_INSTALL_FAILED, strDll);
         else
-            output_write(STRING_UNINSTALL_SUCCESSFUL, strDll);
+            output_write(STRING_UNINSTALL_FAILED, strDll);
+        return -1;
     }
+    if (install)
+        output_write(STRING_INSTALL_SUCCESSFUL, strDll);
+    else
+        output_write(STRING_UNINSTALL_SUCCESSFUL, strDll);
 
     if(DllHandle)
         FreeLibrary(DllHandle);
@@ -320,11 +307,8 @@ int main(int argc, char* argv[])
 
     if (!DllFound)
     {
-        if(!Silent)
-        {
-            output_write(STRING_HEADER);
-            output_write(STRING_USAGE);
-        }
+        output_write(STRING_HEADER);
+        output_write(STRING_USAGE);
         return 1;
     }