ソースの書き方を変更

[YACASL2.git] / src / cerr.c

diff --git a/src/cerr.c b/src/cerr.c
index 973881d..24f620d 100644 (file)
--- a/src/cerr.c
+++ b/src/cerr.c
@@ -1,37 +1,32 @@
+#include <stdio.h>
+#include <stdlib.h>
+#include <string.h>
+#include <assert.h>
+#include <stdbool.h>
 #include "cerr.h"
 
-/* mallocを実行し、メモリを確保できない場合は */
-/* エラーを出力して終了 */
-void *malloc_chk(size_t size, char *tag)
+/**
+ * エラーの初期化
+ */
+void cerr_init()
 {
-    void *p;
-
-    if((p = malloc(size)) == NULL) {
-        fprintf(stderr, "%s: cannot allocate memory\n", tag);
-        exit(-1);
-    }
-    return p;
+    cerr = malloc_chk(sizeof(CERR), "cerr");
+    cerr->num = 0;
 }
 
-/* malloc_chkを実行してメモリを確保してから、 */
-/* コピーした文字列を返す */
-char *strdup_chk(const char *s, char *tag)
-{
-    assert(s != NULL);
-    char *t;
-
-    t = malloc_chk(strlen(s) + 1, tag);
-    strcpy(t, s);
-    return t;
-}
-
-/* 現在のエラー */
+/**
+ * 現在のエラー
+ */
 CERR *cerr;
 
-/* エラーリスト */
+/**
+ * エラーリスト
+ */
 CERRLIST *cerrlist;
 
-/* エラーリストを作成・追加する */
+/**
+ * エラーリストを作成または追加する
+ */
 bool addcerrlist(int newerrc, CERR newerrv[])
 {
     int i;
@@ -47,19 +42,38 @@ bool addcerrlist(int newerrc, CERR newerrv[])
         p = q->next = malloc_chk(sizeof(CERRLIST), "cerrlist.next");
     }
     for(i = 0; i < newerrc; i++) {
-        p->cerr = &(newerrv[i]);
-        p->next = malloc_chk(sizeof(CERRLIST), "cerrlist.next");
+        p->cerr = &newerrv[i];
         q = p;
-        p = p->next;
+        p = p->next = malloc_chk(sizeof(CERRLIST), "cerrlist.next");
     }
     q->next = NULL;
     return true;
 }
 
-/* 現在のエラーを設定する */
+/**
+ * エラーリストを表示する
+ */
+void printcerrlist()
+{
+    CERRLIST *p;
+
+    if(cerrlist == NULL) {
+        puts("error list is null.");
+    } else {
+        for(p = cerrlist; p != NULL; p = p->next) {
+            printf("%d: %s\n", p->cerr->num, p->cerr->msg);
+        }
+    }
+}
+
+/**
+ * 現在のエラーを設定する
+ */
 void setcerr(int num, const char *str)
 {
+    /* 現在のエラー番号を設定  */
     cerr->num = num;
+    /* 現在のエラーメッセージを設定 */
     cerr->msg = malloc_chk(CERRMSGSIZE + 1, "cerr.msg");
     if(str != NULL && strlen(str) <= CERRSTRSIZE) {
         sprintf(cerr->msg, "%s: %s", str, getcerrmsg(cerr->num));
@@ -68,30 +82,37 @@ void setcerr(int num, const char *str)
     }
 }
 
-/* エラーリストから、エラー番号に対応するメッセージを返す */
+/**
+ * エラーリストから、エラー番号に対応するメッセージを返す
+ */
 char *getcerrmsg(int num)
 {
     CERRLIST *p;
+    char *msg = "unknown error";
 
     for(p = cerrlist; p != NULL; p = p->next) {
         if(num == p->cerr->num) {
-            return p->cerr->msg;
+            msg = p->cerr->msg;
+            break;
         }
     }
-    return "unknown error";
+    return msg;
 }
 
-/* エラーリストと現在のエラーを解放する */
+/**
+ * エラーリストと現在のエラーを解放する
+ */
 void freecerr()
 {
     CERRLIST *p = cerrlist, *q;
 
+    /* 現在のエラーメッセージを解放 */
+    free_chk(cerr->msg, "cerr.msg");
+    /* 現在のエラーを解放 */
+    free_chk(cerr, "cerr");
     /* エラーリストを解放 */
-    while(p != NULL) {
+    for(p = cerrlist; p != NULL; p = q) {
         q = p->next;
-        free(p);
-        p = q;
+        free_chk(p, "freecerr.p");
     }
-    /* 現在のエラーを解放 */
-    free(cerr);
 }