Added format_environment() and unformat_environment().

[nssm.git] / gui.cpp

diff --git a/gui.cpp b/gui.cpp
index d9b7011..85a7256 100644 (file)
--- a/gui.cpp
+++ b/gui.cpp
@@ -146,27 +146,15 @@ int nssm_gui(int resource, nssm_service_t *service) {
     }\r
 \r
     if (envlen) {\r
-      /* Replace NULL with CRLF. Leave NULL NULL as the end marker. */\r
-      unsigned long i, j;\r
-      unsigned long newlen = envlen;\r
-      for (i = 0; i < envlen; i++) if (! env[i] && env[i + 1]) newlen++;\r
-\r
-      TCHAR *formatted = (TCHAR *) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, newlen * sizeof(TCHAR));\r
-      if (formatted) {\r
-        for (i = 0, j = 0; i < envlen; i++) {\r
-          formatted[j] = env[i];\r
-          if (! env[i]) {\r
-            if (env[i + 1]) {\r
-              formatted[j] = _T('\r');\r
-              formatted[++j] = _T('\n');\r
-            }\r
-          }\r
-          j++;\r
-        }\r
+      TCHAR *formatted;\r
+      unsigned long newlen;\r
+      if (format_environment(env, envlen, &formatted, &newlen)) {\r
+        popup_message(MB_OK | MB_ICONEXCLAMATION, NSSM_EVENT_OUT_OF_MEMORY, _T("environment"), _T("nssm_dlg()"));\r
+      }\r
+      else {\r
         SetDlgItemText(tablist[NSSM_TAB_ENVIRONMENT], IDC_ENVIRONMENT, formatted);\r
         HeapFree(GetProcessHeap(), 0, formatted);\r
       }\r
-      else popup_message(MB_OK | MB_ICONEXCLAMATION, NSSM_EVENT_OUT_OF_MEMORY, _T("environment"), _T("nssm_dlg()"));\r
     }\r
     if (service->envlen && service->env_extralen) popup_message(MB_OK | MB_ICONWARNING, NSSM_GUI_WARN_ENVIRONMENT);\r
   }\r
@@ -483,28 +471,15 @@ int configure(HWND window, nssm_service_t *service, nssm_service_t *orig_service
       return 5;\r
     }\r
 \r
-    /* Strip CR and replace LF with NULL. */\r
-    unsigned long newlen = 0;\r
-    unsigned long i, j;\r
-    for (i = 0; i < envlen; i++) if (env[i] != _T('\r')) newlen++;\r
-    /* Must end with two NULLs. */\r
-    newlen += 2;\r
-\r
-    TCHAR *newenv = (TCHAR *) HeapAlloc(GetProcessHeap(), HEAP_ZERO_MEMORY, newlen * sizeof(TCHAR));\r
-    if (! newenv) {\r
+    TCHAR *newenv;\r
+    unsigned long newlen;\r
+    if (unformat_environment(env, envlen, &newenv, &newlen)) {\r
       HeapFree(GetProcessHeap(), 0, env);\r
       popup_message(MB_OK | MB_ICONEXCLAMATION, NSSM_EVENT_OUT_OF_MEMORY, _T("environment"), _T("install()"));\r
       cleanup_nssm_service(service);\r
       return 5;\r
     }\r
 \r
-    for (i = 0, j = 0; i < envlen; i++) {\r
-      if (env[i] == _T('\r')) continue;\r
-      if (env[i] == _T('\n')) newenv[j] = _T('\0');\r
-      else newenv[j] = env[i];\r
-      j++;\r
-    }\r
-\r
     HeapFree(GetProcessHeap(), 0, env);\r
     env = newenv;\r
     envlen = newlen;\r