ソースコードの推敲
[YACASL2.git] / src / word.c
index bc51180..e355a67 100644 (file)
@@ -16,6 +16,14 @@ static CERR cerr_word[] = {
     { 116, "out of hex range" },
 };
 
+/**
+ * wordのエラーをエラーリストに追加
+ */
+void addcerrlist_word()
+{
+    addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
+}
+
 /**
  * 10進数の文字列をWORD値に変換
  */
@@ -45,7 +53,7 @@ WORD h2word(const char *str)
 {
     assert(*str == '#');
 
-    WORD word = 0x0;
+    WORD w = 0x0;
     char *check;
     str++;
     if(*str == '-' || strlen(str) > 4) {
@@ -53,12 +61,12 @@ WORD h2word(const char *str)
         return 0;
     }
     /* WORD値に変換 */
-    word = (WORD)strtol(str, &check, 16);
+    w = (WORD)strtol(str, &check, 16);
     if(*check != '\0') {
         setcerr(115, str-1);    /* not hex */
         return 0x0;
     }
-    return word;
+    return w;
 }
 
 /**
@@ -66,21 +74,19 @@ WORD h2word(const char *str)
  */
 WORD nh2word(const char *str)
 {
-    assert(sizeof(WORD)*8 == 16); /* WORD型のサイズが16ビットであることを確認 */
-    addcerrlist(ARRAYSIZE(cerr_word), cerr_word); /* エラーの設定 */
-
-    WORD word;
+    WORD w;
 
+    assert(sizeof(WORD) * 8 == 16); /* WORD型のサイズが16ビットであることを確認 */
     if(!isdigit(*str) && *str != '-' && *str != '#') {
         setcerr(114, str);    /* not integer */
         return 0x0;
     }
     if(*str == '#') {
-        word = h2word(str);
+        w = h2word(str);
     } else {
-        word = n2word(str);
+        w = n2word(str);
     }
-    return word;
+    return w;
 }
 
 /**
@@ -89,9 +95,9 @@ WORD nh2word(const char *str)
 char *word2n(WORD word)
 {
     enum {
-        MAXLEN = 6,        /* WORD値を10進数で表したときの最大桁数 */
+        MAXLEN = 5,        /* WORD値を10進数で表したときの最大桁数 */
     };
-    char *p = malloc_chk(MAXLEN, "word2n.p"), *digit = malloc_chk(MAXLEN, "word2n.digit");
+    char *p = malloc_chk(MAXLEN + 1, "word2n.p"), *digit = malloc_chk(MAXLEN + 1, "word2n.digit");
     int i = 0, j;
 
     do{
@@ -101,7 +107,7 @@ char *word2n(WORD word)
         *(digit + j) = *(p + (i - 1) - j);
     }
     *(digit + j + 1) = '\0';
-    free_chk(p, "word2n.p");
+    FREE(p);
     return digit;
 }