ソースで、コマンド表を2重ポインタからポインタ配列に変更

[YACASL2.git] / src / cmd.c

diff --git a/src/cmd.c b/src/cmd.c
index 1b01d63..04acd32 100644 (file)
--- a/src/cmd.c
+++ b/src/cmd.c
@@ -57,12 +57,14 @@ static int comet2cmdsize = ARRAYSIZE(comet2cmd);
 /**
  * ハッシュ表のサイズ
  */
-static int cmdtabsize;
+enum {
+    CMDTABSIZE = 39,
+};
 
 /**
  * ハッシュ表
  */
-static CMDTAB **cmdtype_code, **code_type;
+static CMDTAB *cmdtype_code[CMDTABSIZE], *code_type[CMDTABSIZE];
 
 /**
  * 命令の名前とタイプからハッシュ値を生成する
@@ -81,10 +83,10 @@ unsigned hash_cmdtype(const char *cmd, CMDTYPE type)
     keys[1]->type = INT;
     keys[1]->val.i = (int)(type & 070);
     /* ハッシュ値の計算 */
-    hashval = hash(2, keys, cmdtabsize);
-    free_chk(keys[0]->val.s, "keys[0].val.s");
-    free_chk(keys[0], "keys[0]");
-    free_chk(keys[1], "keys[1]");
+    hashval = hash(2, keys, CMDTABSIZE);
+    FREE(keys[0]->val.s);
+    FREE(keys[0]);
+    FREE(keys[1]);
     /* ハッシュ値を返す */
     return hashval;
 }
@@ -98,13 +100,11 @@ bool create_cmdtype_code()
     unsigned hashval;
     int i;
 
-    cmdtabsize = comet2cmdsize;                                            /* ハッシュ表のサイズ */
-    cmdtype_code = calloc_chk(cmdtabsize, sizeof(CMDTAB *), "cmdtype_code");
     for(i = 0; i < comet2cmdsize; i++) {
-        p = malloc_chk(sizeof(CMDTAB), "create_cmdtype_code.p");
         hashval = hash_cmdtype(comet2cmd[i].name, comet2cmd[i].type);    /* ハッシュ値の生成 */
-        p->next = cmdtype_code[hashval];                                 /* ハッシュ表に値を追加 */
+        p = malloc_chk(sizeof(CMDTAB), "cmdtype_code");
         p->cmd = &comet2cmd[i];
+        p->next = cmdtype_code[hashval];                                 /* ハッシュ表に値を追加 */
         cmdtype_code[hashval] = p;
     }
     return true;
@@ -137,10 +137,10 @@ void free_cmdtype_code()
     int i;
     CMDTAB *p, *q;
 
-    for(i = 0; i < cmdtabsize; i++) {
+    for(i = 0; i < CMDTABSIZE; i++) {
         for(p = cmdtype_code[i]; p != NULL; p = q) {
             q = p->next;
-            free_chk(p, "free_cmdtype_code");
+            FREE(p);
         }
     }
 }
@@ -151,13 +151,15 @@ void free_cmdtype_code()
 unsigned hash_code(WORD code)
 {
     HKEY *keys[1];
+    unsigned h;
 
     /* 命令コードを設定 */
     keys[0] = malloc_chk(sizeof(HKEY), "hash_code.key");
     keys[0]->type = INT;
     keys[0]->val.i = (int)(code >> 8);
-    /* ハッシュ値を返す */
-    return hash(1, keys, cmdtabsize);
+    h = hash(1, keys, CMDTABSIZE);
+    FREE(keys[0]);
+    return h;
 }
 
 /**
@@ -169,13 +171,11 @@ bool create_code_type()
     unsigned hashval;
     int i;
 
-    cmdtabsize = comet2cmdsize;                    /* ハッシュ表のサイズ */
-    code_type = calloc_chk(comet2cmdsize, sizeof(CMDTAB *), "code_type");
     for(i = 0; i < comet2cmdsize; i++) {
-        p = malloc_chk(sizeof(CMDTAB), "code_type.p");
         hashval = hash_code((&comet2cmd[i])->code);    /* ハッシュ値の生成 */
-        p->next = code_type[hashval];                  /* ハッシュ表に値を追加 */
+        p = malloc_chk(sizeof(CMDTAB), "code_type");
         p->cmd = &comet2cmd[i];
+        p->next = code_type[hashval];                  /* ハッシュ表に値を追加 */
         code_type[hashval] = p;
     }
     return true;
@@ -206,10 +206,10 @@ void free_code_type()
 {
     int i;
     CMDTAB *p, *q;
-    for(i = 0; i < cmdtabsize; i++) {
+    for(i = 0; i < CMDTABSIZE; i++) {
         for(p = code_type[i]; p != NULL; p = q) {
             q = p->next;
-            free_chk(p, "code_type");
+            FREE(p);
         }
     }
 }