Changeset 1519 for trunk/dll/command.c

Timestamp:

May 1, 2010, 9:39:04 PM (15 years ago)

Author:

Gregg Young

Message:

Add ENVIRONMENT_SIZE variable to standardize this size everywhere. Changes to move environment storage to INI file. The environment changes still don't actually get loaded in the new/child process (of course they never did).

File:

• trunk/dll/command.c (modified) (15 diffs)

trunk/dll/command.c

@@ -36 +36 @@
   27 Dec 09 GKY Added QueryCommandSettings to streamline code
   27 Dec 09 GKY Made command hotkeys user selectable.
-  17 JAN 10 GKY Changes to get working with Watcom 1.9 Beta (1/16/10). Mostly cast CHAR CONSTANT * as CHAR *.
+  17 JAN 10 GKY Changes to get working with Watcom 1.9 Beta (1/16/10).
+                Mostly cast CHAR CONSTANT * as CHAR *.
+  01 May 10 GKY Add ENVIRONMENT_SIZE variable to standardize this size everywhere.
+  01 May 10 GKY Changes to move environment storage to INI file
 
 ***********************************************************************/
@@ -77 +80 @@
   PSZ pszCmdLine;
   CHAR title[100];
-  CHAR env[1002];
+  CHAR env[ENVIRONMENT_SIZE];
   ULONG flags;
   ULONG ID;
@@ -375 +378 @@
 
   size = sizeof(BOOL) * 300;
-  PrfQueryProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.UsedCommandIDs", &UsedCommandIDs,
+  PrfQueryProfileData(fmprof, FM3Str, "COMMANDS.UsedCommandIDs", &UsedCommandIDs,
                       &size);
   size = sizeof(BOOL) * 40;
-  PrfQueryProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.UsedHotKeyIDs", &UsedHotKeyIDs,
+  PrfQueryProfileData(fmprof, FM3Str, "COMMANDS.UsedHotKeyIDs", &UsedHotKeyIDs,
                         &size); size = sizeof(BOOL);
-  PrfQueryProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.LoadCommandsFromINI",
+  PrfQueryProfileData(fmprof, FM3Str, "COMMANDS.LoadCommandsFromINI",
                       &fLoadCommandsFromINI, &size);
   if (!fLoadCommandsFromINI) {
@@ -463 +466 @@
   ULONG ID = 0;
   ULONG HotKeyID = 0;
-  CHAR env[1002];
+  CHAR env[ENVIRONMENT_SIZE];
   CHAR key[120];
   CHAR szTitle[100];
@@ -481 +484 @@
       cmdloaded = TRUE;
       size = sizeof(ULONG);
-      PrfQueryProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.SizeSortOrder",
+      PrfQueryProfileData(fmprof, FM3Str, "COMMANDS.SizeSortOrder",
                           &ulSizeCommandsList, &size);
       pszCommandsList = xmallocz(ulSizeCommandsList, pszSrcFile, __LINE__);
       if (pszCommandsList) {
-        PrfQueryProfileString(fmprof, (CHAR *) FM3Str, "COMMANDS.SortOrder",
+        PrfQueryProfileString(fmprof, FM3Str, "COMMANDS.SortOrder",
                               NullStr, pszCommandsList, ulSizeCommandsList);
         p = pszCommandsList;
@@ -502 +505 @@
             sprintf(key, "COMMAND.%sID", szTitle);
             size = sizeof(ULONG);
-            PrfQueryProfileData(fmprof, (CHAR *) FM3Str, key, &ID, &size);
+            PrfQueryProfileData(fmprof, FM3Str, key, &ID, &size);
             sprintf(key, "COMMAND.%sHotKeyID", szTitle);
             size = sizeof(ULONG);
-            PrfQueryProfileData(fmprof, (CHAR *) FM3Str, key, &HotKeyID, &size);
+            PrfQueryProfileData(fmprof, FM3Str, key, &HotKeyID, &size);
             sprintf(key, "COMMAND.%sflags", szTitle);
             size = sizeof(ULONG);
-            PrfQueryProfileData(fmprof, (CHAR *) FM3Str, key, &flags, &size);
+            PrfQueryProfileData(fmprof, FM3Str, key, &flags, &size);
             sprintf(key, "COMMAND.%senv", szTitle);
-            PrfQueryProfileString(fmprof, (CHAR *) FM3Str, key, NullStr, env, sizeof(env));
+            PrfQueryProfileString(fmprof, FM3Str, key, NullStr, env, sizeof(env));
             sprintf(key, "COMMAND.%sCmdLine", szTitle);
-            PrfQueryProfileString(fmprof, (CHAR *) FM3Str, key, NullStr, pszCmdLine, MaxComLineStrg);
+            PrfQueryProfileString(fmprof, FM3Str, key, NullStr, pszCmdLine, MaxComLineStrg);
           }
           info = xmallocz(sizeof(LINKCMDS), pszSrcFile, __LINE__);
@@ -551 +554 @@
         sprintf(key, "COMMAND.%sID", info->title);
         size = sizeof(ULONG);
-        PrfQueryProfileData(fmprof, (CHAR *) FM3Str, key, &ID, &size);
+        PrfQueryProfileData(fmprof, FM3Str, key, &ID, &size);
         sprintf(key, "COMMAND.%sHotKeyID", info->title);
         size = sizeof(ULONG);
-        PrfQueryProfileData(fmprof, (CHAR *) FM3Str, key, &HotKeyID, &size);
+        PrfQueryProfileData(fmprof, FM3Str, key, &HotKeyID, &size);
         sprintf(key, "COMMAND.%senv", info->title);
-        PrfQueryProfileString(fmprof, (CHAR *) FM3Str, key, NullStr, env, sizeof(env));
+        PrfQueryProfileString(fmprof, FM3Str, key, NullStr, env, sizeof(env));
         if (ID != 0) {
           if (env != NullStr)
@@ -585 +588 @@
                      GetPString(IDS_COMMANDSLIMITREACHEDTEXT ));
           }
-          PrfQueryProfileString(fmprof, (CHAR *) FM3Str, info->pszCmdLine, NullStr, env, sizeof(env));
+          PrfQueryProfileString(fmprof, FM3Str, info->pszCmdLine, NullStr, env, sizeof(env));
           info->env = xstrdup(env, pszSrcFile, __LINE__);
         }
@@ -611 +614 @@
     while (info) {
       sprintf(key, "COMMAND.%sflags", info->title);
-      PrfWriteProfileData(fmprof, (CHAR *) FM3Str, key, &info->flags, sizeof(ULONG));
+      PrfWriteProfileData(fmprof, FM3Str, key, &info->flags, sizeof(ULONG));
       sprintf(key, "COMMAND.%sCmdLine", info->title);
-      PrfWriteProfileString(fmprof, (CHAR *) FM3Str, key, info->pszCmdLine);
+      PrfWriteProfileString(fmprof, FM3Str, key, info->pszCmdLine);
       bstripcr(info->title);
       sprintf(key, "COMMAND.%sID", info->title);
-      PrfWriteProfileData(fmprof, (CHAR *) FM3Str, key, &info->ID, sizeof(INT));
+      PrfWriteProfileData(fmprof, FM3Str, key, &info->ID, sizeof(INT));
       sprintf(key, "COMMAND.%sHotKeyID", info->title);
-      PrfWriteProfileData(fmprof, (CHAR *) FM3Str, key, &info->HotKeyID, sizeof(INT));
+      PrfWriteProfileData(fmprof, FM3Str, key, &info->HotKeyID, sizeof(INT));
       if (info->env != NullStr) {
         sprintf(key, "COMMAND.%senv", info->title);
-        PrfWriteProfileString(fmprof, (CHAR *) FM3Str, key, info->env);
+        PrfWriteProfileString(fmprof, FM3Str, key, info->env);
       }
       if ((strlen(pszCommandsList) + strlen(info->title) + 1) > ulSizeCommandsList)
@@ -631 +634 @@
       info = info->next;
     } // while info
-    PrfWriteProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.UsedCommandIDs", &UsedCommandIDs,
+    PrfWriteProfileData(fmprof, FM3Str, "COMMANDS.UsedCommandIDs", &UsedCommandIDs,
                         sizeof(BOOL) * 300);
-    PrfWriteProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.UsedHotKeyIDs", &UsedHotKeyIDs,
+    PrfWriteProfileData(fmprof, FM3Str, "COMMANDS.UsedHotKeyIDs", &UsedHotKeyIDs,
                         sizeof(BOOL) * 40);
     ulSizeCommandsList = strlen(pszCommandsList) + 1;
-    PrfWriteProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.SizeSortOrder",
+    PrfWriteProfileData(fmprof, FM3Str, "COMMANDS.SizeSortOrder",
                         &ulSizeCommandsList, sizeof(ULONG));
-    PrfWriteProfileString(fmprof, (CHAR *) FM3Str, "COMMANDS.SortOrder", pszCommandsList);
-    PrfWriteProfileData(fmprof, (CHAR *) FM3Str, "COMMANDS.LoadCommandsFromINI",
+    PrfWriteProfileString(fmprof, FM3Str, "COMMANDS.SortOrder", pszCommandsList);
+    PrfWriteProfileData(fmprof, FM3Str, "COMMANDS.LoadCommandsFromINI",
                         &fLoadCommandsFromINI, sizeof(BOOL));
 }
@@ -748 +751 @@
     pszCommandsList = xmallocz(ulSizeCommandsList, pszSrcFile, __LINE__);
     if (pszCommandsList)
-      PrfQueryProfileString(fmprof, (CHAR *) FM3Str, "COMMANDS.SortOrder",
+      PrfQueryProfileString(fmprof, FM3Str, "COMMANDS.SortOrder",
                             NullStr, pszCommandsList, ulSizeCommandsList);
     return 0;
@@ -798 +801 @@
    PSZ pszWorkBuf;
    APIRET ret;
-   CHAR env[1002];
+   CHAR env[ENVIRONMENT_SIZE];
    INT x;
 
@@ -825 +828 @@
    temp->flags = CheckExecutibleFlags(hwnd, 3);
    *env = 0;
-   WinQueryDlgItemText(hwnd, CMD_ENVIRON, 1000, env);
+   WinQueryDlgItemText(hwnd, CMD_ENVIRON, ENVIRONMENT_SIZE - 1, env);
    bstripcr(env);
    if (*env)
@@ -1103 +1106 @@
           sprintf(keyHotKeyID, "COMMAND.%sHotKeyID", temp);
           sprintf(keyenv, "COMMAND.%senv", temp);
-          PrfWriteProfileData(fmprof, (CHAR *) FM3Str, keyID, NULL, 0);
-          PrfWriteProfileData(fmprof, (CHAR *) FM3Str, keyHotKeyID, NULL, 0);
-          PrfWriteProfileString(fmprof, (CHAR *) FM3Str, keyenv, NULL);
+          PrfWriteProfileData(fmprof, FM3Str, keyID, NULL, 0);
+          PrfWriteProfileData(fmprof, FM3Str, keyHotKeyID, NULL, 0);
+          PrfWriteProfileString(fmprof, FM3Str, keyenv, NULL);
           x = (SHORT) WinSendDlgItemMsg(hwnd,
                                         CMD_LISTBOX,
@@ -1158 +1161 @@
         sprintf(keyHotKeyID, "COMMAND.%sHotKeyID", temp->title);
         sprintf(keyenv, "COMMAND.%senv", temp->title);
-        PrfQueryProfileData(fmprof, (CHAR *) FM3Str, keyID, &ID, &size);
-        PrfQueryProfileData(fmprof, (CHAR *) FM3Str, keyHotKeyID, &HotKeyID, &size);
+        PrfQueryProfileData(fmprof, FM3Str, keyID, &ID, &size);
+        PrfQueryProfileData(fmprof, FM3Str, keyHotKeyID, &HotKeyID, &size);
         temp->ID = ID;
         if (temp->HotKeyID == HotKeyID || temp->HotKeyID == 0)
           fDontCheckHotKey = TRUE;
         if (kill_command(temp->title)) {
-          PrfWriteProfileData(fmprof, (CHAR *) FM3Str, keyID, NULL, 0);
-          PrfWriteProfileData(fmprof, (CHAR *) FM3Str, keyHotKeyID, NULL, 0);
-          PrfWriteProfileString(fmprof, (CHAR *) FM3Str, keyenv, NULL);
+          PrfWriteProfileData(fmprof, FM3Str, keyID, NULL, 0);
+          PrfWriteProfileData(fmprof, FM3Str, keyHotKeyID, NULL, 0);
+          PrfWriteProfileString(fmprof, FM3Str, keyenv, NULL);
           x = (SHORT) WinSendDlgItemMsg(hwnd,
                                         CMD_LISTBOX,