X-Git-Url: http://j8takagi.net/cgi-bin/gitweb.cgi?p=YACASL2.git;a=blobdiff_plain;f=src%2Flabel.c;h=9124b5cc70a58c276eb2da7fc710ae929e045b92;hp=120223c53908681341d83730f9bcd56ea0a9e675;hb=7f7bde5dd1305cfc598fe09030c69aad0dc01368;hpb=e0de529cf2de73a298288fd294b24b964abdba34

diff --git a/src/label.c b/src/label.c
index 120223c..9124b5c 100644
--- a/src/label.c
+++ b/src/label.c
@@ -1,30 +1,44 @@
-#include "casl2.h"
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <assert.h>
+
+#include "cerr.h"
+#include "cmem.h"
+#include "hash.h"
 #include "assemble.h"
 
-int labelcnt = 0;                    /* ラベル数 */
-LABELTAB *labels[LABELTABSIZE];  /* ラベル表 */
+static int labelcnt = 0;                /* ラベル数 */
+static LABELTAB *labels[LABELTABSIZE];  /* ラベル表 */
 
-/* プログラム名とラベルに対応するハッシュ値を返す */
+/**
+ * プログラム名とラベルに対応するハッシュ値を返す
+ */
 unsigned labelhash(const char *prog, const char *label)
 {
     HKEY *keys[2];
     int i = 0;
+
     if(prog != NULL) {
-        keys[i] = malloc(sizeof(HKEY));
+        keys[i] = malloc_chk(sizeof(HKEY), "labelhash.key[]");
         keys[i]->type = CHARS;
-        keys[i++]->val.s = strdup(prog);
+        keys[i]->val.s = strdup_chk(prog, "labelhash.key[].val");
     }
-    keys[i] = malloc(sizeof(HKEY));
+    keys[i] = malloc_chk(sizeof(HKEY), "labelhash.key[]");
     keys[i]->type = CHARS;
-    keys[i]->val.s = strdup(label);
+    keys[i]->val.s = strdup_chk(label, "labelhash.key[].val");
     /* ハッシュ値を返す */
     return hash(i+1, keys, LABELTABSIZE);
 }
 
-/* ラベル表からアドレスを検索する */
+/**
+ * プログラム名とラベルに対応するアドレスをラベル表から検索する
+ */
 WORD getlabel(const char *prog, const char *label)
 {
+    assert(label != NULL);
     LABELTAB *np;
+
     for(np = labels[labelhash(prog, label)]; np != NULL; np = np->next) {
         if((prog == NULL || (np->prog != NULL && strcmp(prog, np->prog) == 0)) &&
            strcmp(label, np->label) == 0)
@@ -35,56 +49,69 @@ WORD getlabel(const char *prog, const char *label)
     return 0xFFFF;
 }
 
-/* ラベルを表に追加する */
+/**
+ * プログラム名、ラベル、アドレスをラベル表に追加する
+ */
 bool addlabel(const char *prog, const char *label, WORD adr)
 {
+    assert(label != NULL);
     LABELTAB *np;
     unsigned hashval;
-    char *keys[2];
-    int i = 0;
 
+    /* 登録されたラベルを検索。すでに登録されている場合はエラー発生 */
     if(getlabel(prog, label) != 0xFFFF) {
         setcerr(101, label);    /* label already defined */
         return false;
     }
-    np = malloc(sizeof(*np));
-    if(np == NULL || (np->label = strdup(label)) == NULL ||
-       (prog != NULL && (np->prog = strdup(prog)) == NULL))
-    {
-        setcerr(102, NULL);    /* label table is full */
-        return false;
-    }
-    if(prog != NULL) {
-        keys[i++] = strdup(prog);
+    /* メモリを確保 */
+    np = malloc_chk(sizeof(LABELTAB), "labels.next");
+    /* プログラム名を設定 */
+    if(prog == NULL) {
+        np->prog = NULL;
+    } else {
+        np->prog = strdup_chk(prog, "labels.prog");
     }
+    /* ラベルを設定 */
+    np->label = strdup_chk(label, "labels.label");
+    /* アドレスを設定 */
+    np->adr = adr;
+    /* ラベル数を設定 */
     labelcnt++;
-    keys[i] = strdup(label);
+    /* ハッシュ表へ追加 */
     hashval = labelhash(prog, label);
     np->next = labels[hashval];
     labels[hashval] = np;
-    np->adr = adr;
     return true;
 }
 
+/**
+ * ラベルを比較した結果を返す
+ */
 int compare_adr(const void *a, const void *b)
 {
-    return (**(const LABELARRAY **)a).adr - (**(const LABELARRAY **)b).adr;
+    return (**(LABELARRAY **)a).adr - (**(LABELARRAY **)b).adr;
 }
 
-/* ラベル表を表示する */
+/**
+ * ラベル表を表示する
+ */
 void printlabel()
 {
     int i, asize = 0;
-    LABELTAB *np = malloc(sizeof(LABELTAB *));
-    LABELARRAY **ar = malloc(labelcnt * sizeof(LABELARRAY **));
+    LABELTAB *np;
+    LABELARRAY *ar[labelcnt];
 
     for(i = 0; i < LABELTABSIZE; i++) {
         for(np = labels[i]; np != NULL; np = np->next) {
-            ar[asize] = malloc(sizeof(LABELARRAY *));
-            ar[asize]->prog = (np->prog == NULL ? NULL : strdup(np->prog));
-            ar[asize]->label = strdup(np->label);
-            ar[asize]->adr = np->adr;
-            asize++;
+            assert(np->label != NULL);
+            ar[asize] = malloc_chk(sizeof(LABELARRAY), "ar[]");
+            if(np->prog == NULL) {
+                ar[asize]->prog = NULL;
+            } else {
+                ar[asize]->prog = strdup_chk(np->prog, "ar[].prog");
+            }
+            ar[asize]->label = strdup_chk(np->label, "ar[].label");
+            ar[asize++]->adr = np->adr;
         }
     }
     qsort(ar, asize, sizeof(*ar), compare_adr);
@@ -96,17 +123,22 @@ void printlabel()
     }
 }
 
-/* ラベル表を解放する */
+/**
+ * ラベル表を解放する
+ */
 void freelabel()
 {
     int i;
     LABELTAB *np, *nq;
+
     for(i = 0; i < LABELTABSIZE; i++) {
         for(np = labels[i]; np != NULL; np = nq) {
             nq = np->next;
-            free(np->prog);
-            free(np->label);
-            free(np);
+            if(np->prog != NULL) {
+                free_chk(np->prog, "np.prog");
+            }
+            free_chk(np->label, "np.label");
+            free_chk(np, "np");
         }
     }
 }