X-Git-Url: http://j8takagi.net/cgi-bin/gitweb.cgi?p=YACASL2.git;a=blobdiff_plain;f=src%2Fword.c;h=bb564cfd8b0f10dce4c35a7a98b7f9ed0fdb1d05;hp=6e9fb1702910f1e1ea4ff8cee583b453ececf926;hb=86e559d164166966a797a1e5855871d48e087ddd;hpb=9a90bce10b932f8aa7045b9dceb4b99156666920

diff --git a/src/word.c b/src/word.c
index 6e9fb17..bb564cf 100644
--- a/src/word.c
+++ b/src/word.c
@@ -1,122 +1,166 @@
+#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[] = {
-    { 114, "not integer" },
-    { 115, "not hex" },
-    { 116, "out of hex range" },
-};
+/**
+ * @brief 10進数値を表す文字列をWORD値に変換する
+ *
+ * @return WORD値
+ *
+ * @param *str 10進数値を表す文字列
+ */
+WORD n2word(const char *str);
 
-bool addcerrlist_word()
-{
-    return addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
-}
+/**
+ * @brief 16進数の文字列をWORD値に変換する
+ *
+ * @return WORD値
+ *
+ * @param *str 16進数値を表す文字列
+ */
+WORD h2word(const char *str);
 
-/* 10進数の文字列をWORD値に変換 */
 WORD n2word(const char *str)
 {
-    assert(isdigit(*str) || *str == '-');
+    assert(isdigit(str[0]) || str[0] == '-');
 
-    char *check;
+    char *check = NULL;
     int n;
     /* WORD値に変換 */
     n = strtol(str, &check, 10);
-    if(*check != '\0') {
+    if(check[0]) {
         setcerr(114, str);    /* not integer */
         return 0x0;
     }
-    /* nが-32768〜32767の範囲にないときは、その下位16ビットを格納 */
+    /* nが-32768から32767の範囲にないときは、その下位16ビットを格納 */
     if(n < -32768 || n > 32767) {
-        n = n & 0xFFFF;
+        n &= 0xFFFF;
     }
     return (WORD)n;
 }
 
-/* 16進数の文字列をWORD値に変換 */
 WORD h2word(const char *str)
 {
-    assert(*str == '#');
+    assert(str[0] == '#');
 
-    WORD word = 0x0;
-    char *check;
+    WORD w = 0;
+    char *check = NULL;
     str++;
     if(*str == '-' || strlen(str) > 4) {
         setcerr(116, str-1);    /* out of hex range */
-        return 0;
+        return 0x0;
     }
     /* WORD値に変換 */
-    word = (WORD)strtol(str, &check, 16);
-    if(*check != '\0') {
+    w = (WORD)strtol(str, &check, 16);
+    if(check[0]) {
         setcerr(115, str-1);    /* not hex */
         return 0x0;
     }
-    return word;
+    return w;
+}
+
+/**
+ * @brief wordのエラー定義
+ */
+static CERR cerr_word[] = {
+    { 114, "not integer" },
+    { 115, "not hex" },
+    { 116, "out of hex range" },
+};
+
+/**
+ * @brief ファイル読み込みのエラー定義
+ */
+static CERR cerr_load[] = {
+    { 210, "load - memory overflow" },
+    { 211, "object file not specified" },
+    { 212, "invalid option" },
+    { 213, "invalid argument" },
+};
+
+/* word.hで定義された関数群 */
+void addcerrlist_load()
+{
+    addcerrlist(ARRAYSIZE(cerr_load), cerr_load);
+}
+
+void addcerrlist_word()
+{
+    addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
 }
 
-/* 10進数または16進数の文字列をWORD値に変換 */
 WORD nh2word(const char *str)
 {
-    addcerrlist_word();
-    WORD word;
+    WORD w;
+
+    assert(sizeof(WORD) * 8 == 16); /* WORD型のサイズが16ビットであることを確認 */
+    if(str == NULL) {
+        return 0x0;
+    }
     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;
 }
 
-/* WORD値を10進数の文字列に変換 */
 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 *n = malloc_chk(MAXLEN + 1, "word2n.n"), tmp;
     int i = 0, j;
 
     do{
-        *(p + i++) = word % 10 + '0';
+        n[i++] = word % 10 + '0';
     } while((word /= 10) > 0);
     for(j = 0; j < i; j++) {
-        *(digit + j) = *(p + (i - 1) - j);
+        tmp = n[j];
+        n[j] = n[(i-1)-j];
+        n[(i-1)-j] = tmp;
     }
-    *(digit + j + 1) = '\0';
-    free_chk(p, "word2n.p");
-    return digit;
+    n[j] = '\0';
+    return n;
 }
 
-/* WORD値を2進数の文字列に変換 */
 char *word2bit(const WORD word)
 {
     enum {
         MAXLEN = 16,        /* WORD値を2進数で表したときの最大桁数 */
     };
     WORD mask = 0x8000;
-    char *bit, *p;
+    char *bit = NULL;
+    int i = 0;
 
-    bit = p = malloc_chk(MAXLEN + 1, "word2bit.bit");
+    bit = malloc_chk(MAXLEN + 1, "word2bit.bit");
     do {
-        *p++ = (word & mask) ? '1' : '0';
+        bit[i++] = (word & mask) ? '1' : '0';
     } while((mask >>= 1) > 0);
-    *p = '\0';
+    bit[i] = '\0';
     return bit;
 }
 
-/* WORD値を解析して表示 */
 void print_dumpword(WORD word, bool logicalmode)
 {
+    char *bit = NULL;
+
     if(logicalmode == true) {
         fprintf(stdout, "%6d", word);
     } else {
         fprintf(stdout, "%6d", (signed short)word);
     }
-    fprintf(stdout, " = #%04X = %s", word, word2bit(word));
+    fprintf(stdout, " = #%04X = %s", word, (bit = word2bit(word)));
     /* 「文字の組」の符号表に記載された文字と、改行（CR）／タブを表示 */
     if(word >= 0x20 && word <= 0x7E) {
         fprintf(stdout, " = \'%c\'", word);
@@ -125,5 +169,5 @@ void print_dumpword(WORD word, bool logicalmode)
     } else if(word == '\t') {
         fprintf(stdout, " = \'\\t\'");
     }
-    fprintf(stdout, "\n");
+    FREE(bit);
 }