hyung-hwan/qse
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

simplified some null checks

Browse Source
This commit is contained in:
hyung-hwan 2015-08-13 02:28:22 +00:00
parent 1963e62006
commit b00eaada24
1 changed files with 16 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

32
qse/lib/xli/xli.c
View File

@ -185,8 +185,7 @@ void* qse_xli_allocmem (qse_xli_t* xli, qse_size_t size)
void* ptr; void* ptr;
ptr = QSE_MMGR_ALLOC (xli->mmgr, size); ptr = QSE_MMGR_ALLOC (xli->mmgr, size);
if (ptr == QSE_NULL) if (!ptr) qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL);
qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL);
return ptr; return ptr;
} }
@ -195,8 +194,7 @@ void* qse_xli_callocmem (qse_xli_t* xli, qse_size_t size)
void* ptr; void* ptr;
ptr = QSE_MMGR_ALLOC (xli->mmgr, size); ptr = QSE_MMGR_ALLOC (xli->mmgr, size);
if (ptr == QSE_NULL) if (!ptr) qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL);
qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL);
else QSE_MEMSET (ptr, 0, size); else QSE_MEMSET (ptr, 0, size);
return ptr; return ptr;
} }
@ -256,7 +254,7 @@ static qse_xli_pair_t* insert_pair (
((key->len + 1) * QSE_SIZEOF(*key->ptr)) + ((key->len + 1) * QSE_SIZEOF(*key->ptr)) +
((alen + 1) * QSE_SIZEOF(*alias->ptr)) + ((alen + 1) * QSE_SIZEOF(*alias->ptr)) +
((tlen + 1) * QSE_SIZEOF(*keytag->ptr))); ((tlen + 1) * QSE_SIZEOF(*keytag->ptr)));
if (pair == QSE_NULL) return QSE_NULL; if (!pair) return QSE_NULL;
kptr = (qse_char_t*)((qse_uint8_t*)(pair + 1) + xli->opt.pair_xtnsize); kptr = (qse_char_t*)((qse_uint8_t*)(pair + 1) + xli->opt.pair_xtnsize);
qse_strcpy (kptr, key->ptr); qse_strcpy (kptr, key->ptr);
@ -318,11 +316,11 @@ qse_xli_pair_t* qse_xli_insertpairwithemptylist (
qse_xli_pair_t* tmp; qse_xli_pair_t* tmp;
val = qse_xli_callocmem (xli, QSE_SIZEOF(*val)); val = qse_xli_callocmem (xli, QSE_SIZEOF(*val));
if (val == QSE_NULL) return QSE_NULL; if (!val) return QSE_NULL;
val->type = QSE_XLI_LIST; val->type = QSE_XLI_LIST;
tmp = qse_xli_insertpair (xli, parent, peer, key, alias, keytag, (qse_xli_val_t*)val); tmp = qse_xli_insertpair (xli, parent, peer, key, alias, keytag, (qse_xli_val_t*)val);
if (tmp == QSE_NULL) qse_xli_freemem (xli, val); if (!tmp) qse_xli_freemem (xli, val);
return tmp; return tmp;
} }
@ -339,7 +337,7 @@ qse_xli_pair_t* qse_xli_insertpairwithstr (
if (strtag) reqlen += (qse_strlen (strtag) + 1) * QSE_SIZEOF(*strtag); if (strtag) reqlen += (qse_strlen (strtag) + 1) * QSE_SIZEOF(*strtag);
val = qse_xli_callocmem (xli, reqlen); val = qse_xli_callocmem (xli, reqlen);
if (val == QSE_NULL) return QSE_NULL; if (!val) return QSE_NULL;
val->type = QSE_XLI_STR; val->type = QSE_XLI_STR;
@ -354,7 +352,7 @@ qse_xli_pair_t* qse_xli_insertpairwithstr (
} }
tmp = qse_xli_insertpair (xli, parent, peer, key, alias, keytag, (qse_xli_val_t*)val); tmp = qse_xli_insertpair (xli, parent, peer, key, alias, keytag, (qse_xli_val_t*)val);
if (tmp == QSE_NULL) qse_xli_freemem (xli, val); if (!tmp) qse_xli_freemem (xli, val);
return tmp; return tmp;
} }
@ -374,13 +372,13 @@ qse_xli_pair_t* qse_xli_insertpairwithstrs (
} }
tmp = qse_xli_insertpairwithstr (xli, parent, peer, key, alias, keytag, &value[0], QSE_NULL); tmp = qse_xli_insertpairwithstr (xli, parent, peer, key, alias, keytag, &value[0], QSE_NULL);
if (tmp == QSE_NULL) return QSE_NULL; if (!tmp) return QSE_NULL;
str = (qse_xli_str_t*)tmp->val; str = (qse_xli_str_t*)tmp->val;
for (i = 1; i < count; i++) for (i = 1; i < count; i++)
{ {
str = qse_xli_addsegtostr (xli, str, QSE_NULL, &value[i]); str = qse_xli_addsegtostr (xli, str, QSE_NULL, &value[i]);
if (str == QSE_NULL) if (!str)
{ {
free_atom (xli->root, (qse_xli_atom_t*)tmp); free_atom (xli->root, (qse_xli_atom_t*)tmp);
return QSE_NULL; return QSE_NULL;
@ -399,7 +397,7 @@ qse_xli_text_t* qse_xli_inserttext (
slen = qse_strlen (str); slen = qse_strlen (str);
text = qse_xli_callocmem (xli, QSE_SIZEOF(*text) + ((slen + 1) * QSE_SIZEOF(*str))); text = qse_xli_callocmem (xli, QSE_SIZEOF(*text) + ((slen + 1) * QSE_SIZEOF(*str)));
if (text == QSE_NULL) return QSE_NULL; if (!text) return QSE_NULL;
text->type = QSE_XLI_TEXT; text->type = QSE_XLI_TEXT;
text->ptr = (const qse_char_t*)(text + 1); text->ptr = (const qse_char_t*)(text + 1);
@ -420,7 +418,7 @@ qse_xli_file_t* qse_xli_insertfile (
plen = qse_strlen (path); plen = qse_strlen (path);
file = qse_xli_callocmem (xli, QSE_SIZEOF(*file) + ((plen + 1) * QSE_SIZEOF(*path))); file = qse_xli_callocmem (xli, QSE_SIZEOF(*file) + ((plen + 1) * QSE_SIZEOF(*path)));
if (file == QSE_NULL) return QSE_NULL; if (!file) return QSE_NULL;
file->type = QSE_XLI_FILE; file->type = QSE_XLI_FILE;
file->path = (const qse_char_t*)(file + 1); file->path = (const qse_char_t*)(file + 1);
@ -438,7 +436,7 @@ qse_xli_eof_t* qse_xli_inserteof (
qse_xli_eof_t* eof; qse_xli_eof_t* eof;
eof = qse_xli_callocmem (xli, QSE_SIZEOF(*eof)); eof = qse_xli_callocmem (xli, QSE_SIZEOF(*eof));
if (eof == QSE_NULL) return QSE_NULL; if (!eof) return QSE_NULL;
eof->type = QSE_XLI_EOF; eof->type = QSE_XLI_EOF;
insert_atom (xli, parent, peer, (qse_xli_atom_t*)eof); insert_atom (xli, parent, peer, (qse_xli_atom_t*)eof);
@ -453,7 +451,7 @@ static qse_xli_root_list_t* make_root (qse_xli_t* xli)
qse_xli_root_list_t* tmp; qse_xli_root_list_t* tmp;
tmp = QSE_MMGR_ALLOC (xli->mmgr, QSE_SIZEOF(*tmp) + xli->opt.root_xtnsize); tmp = QSE_MMGR_ALLOC (xli->mmgr, QSE_SIZEOF(*tmp) + xli->opt.root_xtnsize);
if (tmp == QSE_NULL) if (!tmp)
{ {
qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL); qse_xli_seterrnum (xli, QSE_XLI_ENOMEM, QSE_NULL);
return QSE_NULL; return QSE_NULL;
@ -472,7 +470,9 @@ static void free_val (qse_xli_root_list_t* root, qse_xli_val_t* val)
if ((qse_xli_nil_t*)val != &root->xnil) if ((qse_xli_nil_t*)val != &root->xnil)
{ {
if (val->type == QSE_XLI_LIST) if (val->type == QSE_XLI_LIST)
{
free_list (root, (qse_xli_list_t*)val); free_list (root, (qse_xli_list_t*)val);
}
else if (val->type == QSE_XLI_STR) else if (val->type == QSE_XLI_STR)
{ {
qse_xli_str_t* cur, * next; qse_xli_str_t* cur, * next;
@ -549,7 +549,7 @@ qse_xli_list_t* qse_xli_yieldroot (qse_xli_t* xli)
qse_xli_root_list_t* tmp, * tmp2; qse_xli_root_list_t* tmp, * tmp2;
tmp = make_root (xli); tmp = make_root (xli);
if (tmp == QSE_NULL) return QSE_NULL; if (!tmp) return QSE_NULL;
tmp2 = xli->root; tmp2 = xli->root;
xli->root = tmp; xli->root = tmp;