実行時、バイナリからCOMET II以外の値を読み込んだ場合はエラーが発生するよう変更

[YACASL2.git] / src / cmd.c

diff --git a/src/cmd.c b/src/cmd.c
index add169a..53d3944 100644 (file)
--- a/src/cmd.c
+++ b/src/cmd.c
@@ -1,6 +1,14 @@
-#include "casl2.h"
-
-CMD comet2cmd[] = {
+#include <stdio.h>
+#include <assert.h>
+#include <string.h>
+#include "hash.h"
+#include "struct.h"
+#include "cmem.h"
+
+/**
+ * 機械語命令のリスト
+ */
+static CMD comet2cmd[] = {
     { "NOP", NONE, 0x0 },
     { "LD", R_ADR_X_, 0x1000 },
     { "ST", R_ADR_X, 0x1100 },
@@ -41,20 +49,34 @@ CMD comet2cmd[] = {
     { "RET", NONE, 0x8100 },
 };
 
-int comet2cmdsize = ARRAYSIZE(comet2cmd);
-int cmdtabsize;
-CMDTAB **cmdtype_code, **code_type;
-
-/* 命令と命令タイプからハッシュ値を生成する */
-unsigned hash_cmdtype(const char *cmd, CMDTYPE type) {
+/**
+ * 命令コード配列のサイズ
+ */
+static int comet2cmdsize = ARRAYSIZE(comet2cmd);
+
+/**
+ * 命令表のサイズ
+ */
+static int cmdtabsize;
+
+/**
+ * ハッシュ表
+ */
+static CMDTAB **cmdtype_code, **code_type;
+
+/**
+ * 命令の名前とタイプからハッシュ値を生成する
+ */
+unsigned hash_cmdtype(const char *cmd, CMDTYPE type)
+{
     HKEY *keys[2];
     unsigned hashval;
 
-    /* 命令をセット */
+    /* 命令名を設定 */
     keys[0] = malloc_chk(sizeof(HKEY), "hash_cmdtype.keys[0]");
     keys[0]->type = CHARS;
     keys[0]->val.s = strdup_chk(cmd, "keys[0].val.s");
-    /* 命令タイプをセット */
+    /* 命令タイプを設定 */
     keys[1] = malloc_chk(sizeof(HKEY), "hash_cmdtype.keys[1]");
     keys[1]->type = INT;
     keys[1]->val.i = (int)(type & 070);
@@ -67,7 +89,9 @@ unsigned hash_cmdtype(const char *cmd, CMDTYPE type) {
     return hashval;
 }
 
-/* 命令と命令タイプがキーのハッシュ表を作成する */
+/**
+ * 名前とタイプがキーの命令ハッシュ表を作成する
+ */
 bool create_cmdtype_code()
 {
     CMDTAB *np;
@@ -80,7 +104,7 @@ bool create_cmdtype_code()
         *(cmdtype_code + i) = NULL;
     }
     for(i = 0; i < comet2cmdsize; i++) {
-        np = malloc_chk(sizeof(CMDTAB), "cmdtype_code.next");
+        np = malloc_chk(sizeof(CMDTAB), "cmdtype_code.np");
         np->cmd = NULL;
         np->next = NULL;
         /* ハッシュ値の生成 */
@@ -93,8 +117,10 @@ bool create_cmdtype_code()
     return true;
 }
 
-/* 命令と命令タイプから、命令コードを取得する */
-/* 無効な場合は0xFFFFを返す */
+/**
+ * 命令の名前とタイプから、命令コードを返す
+ * 無効な場合は0xFFFFを返す
+ */
 WORD getcmdcode(const char *cmd, CMDTYPE type)
 {
     CMDTAB *np;
@@ -108,7 +134,9 @@ WORD getcmdcode(const char *cmd, CMDTYPE type)
     return 0xFFFF;
 }
 
-/* 命令と命令タイプがキーのハッシュ表を解放する */
+/**
+ * 名前とタイプがキーの命令ハッシュ表を解放する
+ */
 void free_cmdtype_code()
 {
     int i;
@@ -125,12 +153,14 @@ void free_cmdtype_code()
     free_chk(cmdtype_code, "cmdtype_code");
 }
 
-/* 命令コードからハッシュ値を生成する */
+/**
+ * 命令コードからハッシュ値を生成する
+ */
 unsigned hash_code(WORD code)
 {
     HKEY *keys[1];
 
-    /* 命令コードをセット */
+    /* 命令コードを設定 */
     keys[0] = malloc_chk(sizeof(HKEY), "hash_code.key");
     keys[0]->type = INT;
     keys[0]->val.i = (int)(code >> 8);
@@ -138,7 +168,9 @@ unsigned hash_code(WORD code)
     return hash(1, keys, cmdtabsize);
 }
 
-/* 命令コードがキーのハッシュ表を作成する */
+/**
+ * コードがキーの命令ハッシュ表を作成する
+ */
 bool create_code_type()
 {
     CMDTAB *np;
@@ -151,7 +183,7 @@ bool create_code_type()
         *(code_type + i) = NULL;
     }
     for(i = 0; i < comet2cmdsize; i++) {
-        np = malloc_chk(sizeof(CMDTAB), "code_type.next");
+        np = malloc_chk(sizeof(CMDTAB), "code_type.np");
         np->cmd = NULL;
         np->next = NULL;
         /* ハッシュ値の生成 */
@@ -164,8 +196,10 @@ bool create_code_type()
     return true;
 }
 
-/* 命令コードから命令タイプを取得する */
-/* 無効な場合はNONEを返す */
+/**
+ * 命令コードから命令タイプを返す
+ * 無効な場合はNONEを返す
+ */
 CMDTYPE getcmdtype(WORD code)
 {
     CMDTAB *np;
@@ -174,10 +208,12 @@ CMDTYPE getcmdtype(WORD code)
             return np->cmd->type;
         }
     }
-    return NONE;
+    return NOTCMD;
 }
 
-/* 命令コードがキーのハッシュ表を解放する */
+/**
+ * コードがキーの命令ハッシュ表を解放する
+ */
 void free_code_type()
 {
     int i;
@@ -186,9 +222,9 @@ void free_code_type()
         np = code_type[i];
         while(np != NULL) {
             nq = np->next;
-            free(np);
+            free_chk(np, "code_type.np");
             np = nq;
         }
     }
-    free(code_type);
+    free_chk(code_type, "code_type");
 }