バージョンアップ

[YACASL2.git] / src / cerr.c

diff --git a/src/cerr.c b/src/cerr.c
index d595275..455ab5b 100644 (file)
--- a/src/cerr.c
+++ b/src/cerr.c
@@ -1,57 +1,37 @@
-#include <stdio.h>
-#include <stdlib.h>
-#include <string.h>
-#include <assert.h>
-#include <stdbool.h>
 #include "cerr.h"
 
-/**
- * エラーの初期化
- */
 void cerr_init()
 {
     cerr = malloc_chk(sizeof(CERR), "cerr");
     cerr->num = 0;
 }
 
-/**
- * 現在のエラー
- */
-CERR *cerr;
+CERR *cerr = NULL;
 
-/**
- * エラーリスト
- */
 CERRLIST *cerrlist = NULL;
 
-/**
- * エラーリストを作成または追加する
- */
-void addcerrlist(int newerrc, CERR newerrv[])
+void addcerrlist(int cerrc, CERR cerrv[])
 {
-    int i;
-    CERRLIST *p = NULL, *q;
+    CERRLIST *stat = NULL;
+    CERRLIST *p = NULL;
 
-    assert(newerrc > 0 && newerrv != NULL);
-    for(i = 0; i < newerrc; i++) {
+    assert(cerrc > 0 && cerrv != NULL);
+    for(int i = 0; i < cerrc; i++) {
         if(p == NULL) {
-            p = q = malloc_chk(sizeof(CERRLIST), "cerrlist");
+            stat = p = malloc_chk(sizeof(CERRLIST), "cerrlist");
         } else {
-            p = p->next = malloc_chk(sizeof(CERRLIST), "cerrlist.next");
+            p = p->next = malloc_chk(sizeof(CERRLIST), "cerrlist->next");
         }
-        p->cerr = &newerrv[i];
+        p->cerr = &cerrv[i];
         p->next = NULL;
     }
     p->next = cerrlist;
-    cerrlist = q;
+    cerrlist = stat;
 }
 
-/**
- * エラーリストを表示する
- */
 void printcerrlist()
 {
-    CERRLIST *p;
+    CERRLIST *p = NULL;
 
     if(cerrlist == NULL) {
         puts("error list is null.");
@@ -62,28 +42,22 @@ void printcerrlist()
     }
 }
 
-/**
- * 現在のエラーを設定する
- */
 void setcerr(int num, const char *str)
 {
     /* 現在のエラー番号を設定  */
     cerr->num = num;
     /* 現在のエラーメッセージを設定 */
     cerr->msg = malloc_chk(CERRMSGSIZE + 1, "cerr.msg");
-    if(str != NULL && strlen(str) <= CERRSTRSIZE) {
+    if(0 < strlen(str) && strlen(str) <= CERRSTRSIZE) {
         sprintf(cerr->msg, "%s: %s", str, getcerrmsg(cerr->num));
     } else {
         strcpy(cerr->msg, getcerrmsg(cerr->num));
     }
 }
 
-/**
- * エラーリストから、エラー番号に対応するメッセージを返す
- */
 char *getcerrmsg(int num)
 {
-    CERRLIST *p;
+    CERRLIST *p = NULL;
     char *msg = "unknown error";
 
     for(p = cerrlist; p != NULL; p = p->next) {
@@ -95,12 +69,10 @@ char *getcerrmsg(int num)
     return msg;
 }
 
-/**
- * エラーリストと現在のエラーを解放する
- */
 void freecerr()
 {
-    CERRLIST *p = cerrlist, *q;
+    CERRLIST *p = NULL;
+    CERRLIST *q = NULL;
 
     /* 現在のエラーメッセージを解放 */
     FREE(cerr->msg);