projects
/
YACASL2.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Valgrindで見つかったメモリー周辺の問題点を修正
[YACASL2.git]
/
src
/
word.c
diff --git
a/src/word.c
b/src/word.c
index
9371758
..
925946b
100644
(file)
--- a/
src/word.c
+++ b/
src/word.c
@@
-1,18
+1,32
@@
+#include <stdio.h>
+#include <string.h>
+#include <assert.h>
+#include <stdlib.h>
+#include <ctype.h>
+
#include "word.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" },
};
{ 114, "not integer" },
{ 115, "not hex" },
{ 116, "out of hex range" },
};
-bool addcerrlist_word()
+/**
+ * wordのエラーをエラーリストに追加
+ */
+void addcerrlist_word()
{
{
-
return
addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
+ addcerrlist(ARRAYSIZE(cerr_word), cerr_word);
}
}
-/* 10進数の文字列をWORD値に変換 */
+/**
+ * 10進数の文字列をWORD値に変換
+ */
WORD n2word(const char *str)
{
assert(isdigit(*str) || *str == '-');
WORD n2word(const char *str)
{
assert(isdigit(*str) || *str == '-');
@@
-25,14
+39,16
@@
WORD n2word(const char *str)
setcerr(114, str); /* not integer */
return 0x0;
}
setcerr(114, str); /* not integer */
return 0x0;
}
- /* nã
\81
\8c
-32768ã
\80
\9c
32767の範囲にないときは、その下位16ビットを格納 */
+ /* nã
\81
\8c
-32768ã
\81
\8b
ã
\82
\89
32767の範囲にないときは、その下位16ビットを格納 */
if(n < -32768 || n > 32767) {
n = n & 0xFFFF;
}
return (WORD)n;
}
if(n < -32768 || n > 32767) {
n = n & 0xFFFF;
}
return (WORD)n;
}
-/* 16進数の文字列をWORD値に変換 */
+/**
+ * 16進数の文字列をWORD値に変換
+ */
WORD h2word(const char *str)
{
assert(*str == '#');
WORD h2word(const char *str)
{
assert(*str == '#');
@@
-53,11
+69,15
@@
WORD h2word(const char *str)
return word;
}
return word;
}
-/* 10進数または16進数の文字列をWORD値に変換 */
+/**
+ * 10進数または16進数の文字列をWORD値に変換
+ */
WORD nh2word(const char *str)
{
WORD nh2word(const char *str)
{
- addcerrlist_word();
+ assert(sizeof(WORD)*8 == 16); /* WORD型のサイズが16ビットであることを確認 */
+
WORD word;
WORD word;
+
if(!isdigit(*str) && *str != '-' && *str != '#') {
setcerr(114, str); /* not integer */
return 0x0;
if(!isdigit(*str) && *str != '-' && *str != '#') {
setcerr(114, str); /* not integer */
return 0x0;
@@
-70,27
+90,40
@@
WORD nh2word(const char *str)
return word;
}
return word;
}
-/* WORD値を10進数の文字列に変換 */
+/**
+ * WORD値を10進数の文字列に変換
+ */
char *word2n(WORD word)
{
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;
int i = 0, j;
+
do{
*(p + i++) = word % 10 + '0';
} while((word /= 10) > 0);
for(j = 0; j < i; 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(p);
+ return digit;
}
}
-/* WORD値を2進数の文字列に変換 */
+/**
+ * WORD値を2進数の文字列に変換
+ */
char *word2bit(const WORD word)
{
char *word2bit(const WORD word)
{
+ enum {
+ MAXLEN = 16, /* WORD値を2進数で表したときの最大桁数 */
+ };
WORD mask = 0x8000;
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);
do {
*p++ = (word & mask) ? '1' : '0';
} while((mask >>= 1) > 0);
@@
-98,7
+131,9
@@
char *word2bit(const WORD word)
return bit;
}
return bit;
}
-/* WORD値を解析して表示 */
+/**
+ * WORD値を解析して表示
+ */
void print_dumpword(WORD word, bool logicalmode)
{
if(logicalmode == true) {
void print_dumpword(WORD word, bool logicalmode)
{
if(logicalmode == true) {