X-Git-Url: http://j8takagi.net/cgi-bin/gitweb.cgi?a=blobdiff_plain;f=src%2Fword.c;h=bc511807d68a3535ca6c602b331cfed219571f2c;hb=d644fa5efb63608e59b9f7b84c7806c91c970b5d;hp=93717580ec69e9ac21e03c7dfd1028965c5de8c6;hpb=4ee27a568fb9222907a566e59aaefe248f08a8e4;p=YACASL2.git

diff --git a/src/word.c b/src/word.c
index 9371758..bc51180 100644
--- a/src/word.c
+++ b/src/word.c
@@ -1,18 +1,24 @@
+#include <stdio.h>
+#include <string.h>
+#include <assert.h>
+#include <stdlib.h>
+#include <ctype.h>
+
 #include "word.h"
+#include "cerr.h"
 
-/* wordのエラー定義 */
-CERR cerr_word[] = {
+/**
+ * wordのエラー定義
+ */
+static CERR cerr_word[] = {
     { 114, "not integer" },
     { 115, "not hex" },
     { 116, "out of hex range" },
 };
 
-bool addcerrlist_word()
-{
-    return addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
-}
-
-/* 10進数の文字列をWORD値に変換 */
+/**
+ * 10進数の文字列をWORD値に変換
+ */
 WORD n2word(const char *str)
 {
     assert(isdigit(*str) || *str == '-');
@@ -25,14 +31,16 @@ WORD n2word(const char *str)
         setcerr(114, str);    /* not integer */
         return 0x0;
     }
-    /* nが-32768〜32767の範囲にないときは、その下位16ビットを格納 */
+    /* nが-32768から32767の範囲にないときは、その下位16ビットを格納 */
     if(n < -32768 || n > 32767) {
         n = n & 0xFFFF;
     }
     return (WORD)n;
 }
 
-/* 16進数の文字列をWORD値に変換 */
+/**
+ * 16進数の文字列をWORD値に変換
+ */
 WORD h2word(const char *str)
 {
     assert(*str == '#');
@@ -53,11 +61,16 @@ WORD h2word(const char *str)
     return word;
 }
 
-/* 10進数または16進数の文字列をWORD値に変換 */
+/**
+ * 10進数または16進数の文字列をWORD値に変換
+ */
 WORD nh2word(const char *str)
 {
-    addcerrlist_word();
+    assert(sizeof(WORD)*8 == 16); /* WORD型のサイズが16ビットであることを確認 */
+    addcerrlist(ARRAYSIZE(cerr_word), cerr_word); /* エラーの設定 */
+
     WORD word;
+
     if(!isdigit(*str) && *str != '-' && *str != '#') {
         setcerr(114, str);    /* not integer */
         return 0x0;
@@ -70,27 +83,40 @@ WORD nh2word(const char *str)
     return word;
 }
 
-/* WORD値を10進数の文字列に変換 */
+/**
+ * WORD値を10進数の文字列に変換
+ */
 char *word2n(WORD word)
 {
-    char *p = malloc_chk(6, "word2n.p"), *q = malloc_chk(6, "word2n.q");
+    enum {
+        MAXLEN = 6,        /* WORD値を10進数で表したときの最大桁数 */
+    };
+    char *p = malloc_chk(MAXLEN, "word2n.p"), *digit = malloc_chk(MAXLEN, "word2n.digit");
     int i = 0, j;
+
     do{
         *(p + i++) = word % 10 + '0';
     } while((word /= 10) > 0);
     for(j = 0; j < i; j++) {
-        *(q + j) = *(p + (i - 1) - j);
+        *(digit + j) = *(p + (i - 1) - j);
     }
-    *(q + j + 1) = '\0';
-    return q;
+    *(digit + j + 1) = '\0';
+    free_chk(p, "word2n.p");
+    return digit;
 }
 
-/* WORD値を2進数の文字列に変換 */
+/**
+ * WORD値を2進数の文字列に変換
+ */
 char *word2bit(const WORD word)
 {
+    enum {
+        MAXLEN = 16,        /* WORD値を2進数で表したときの最大桁数 */
+    };
     WORD mask = 0x8000;
-    char *bit = malloc_chk(16 + 1, "word2bit.bit"), *p;
-    p = bit;
+    char *bit, *p;
+
+    bit = p = malloc_chk(MAXLEN + 1, "word2bit.bit");
     do {
         *p++ = (word & mask) ? '1' : '0';
     } while((mask >>= 1) > 0);
@@ -98,7 +124,9 @@ char *word2bit(const WORD word)
     return bit;
 }
 
-/* WORD値を解析して表示 */
+/**
+ * WORD値を解析して表示
+ */
 void print_dumpword(WORD word, bool logicalmode)
 {
     if(logicalmode == true) {