ShuriZma
/
xemu
mirror of https://github.com/xemu-project/xemu.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

libcacard: introduce NEXT_TOKEN macro 

vcard_emul_options now has repetitive code to read the current
token and advance to the next. After the previous changes,
this repetitive code can be moved in a NEXT_TOKEN macro to
avoid having this code duplicated.

Signed-off-by: Christophe Fergeau <cfergeau@redhat.com>
Reviewed-by: Alon Levy <alevy@redhat.com>
This commit is contained in:
Christophe Fergeau 2011-07-22 13:42:19 +02:00 committed by Alon Levy
parent a5aa842a05
commit d246b3cfd5
1 changed files with 24 additions and 47 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

71
libcacard/vcard_emul_nss.c
View File

@ -1105,6 +1105,26 @@ find_blank(const char *str)
static VCardEmulOptions options; static VCardEmulOptions options;
#define READER_STEP 4 #define READER_STEP 4
/* Expects "args" to be at the beginning of a token (ie right after the ','
* ending the previous token), and puts the next token start in "token",
* and its length in "token_length". "token" will not be nul-terminated.
* After calling the macro, "args" will be advanced to the beginning of
* the next token.
* This macro may call continue or break.
*/
#define NEXT_TOKEN(token) \
(token) = args; \
args = strpbrk(args, ",)"); \
if (*args == 0) { \
break; \
} \
if (*args == ')') { \
args++; \
continue; \
} \
(token##_length) = args - (token); \
args = strip(args+1);
VCardEmulOptions * VCardEmulOptions *
vcard_emul_options(const char *args) vcard_emul_options(const char *args)
{ {
@ -1140,58 +1160,15 @@ vcard_emul_options(const char *args)
} }
args = strip(args+1); args = strip(args+1);
name = args; NEXT_TOKEN(name)
args = strpbrk(args, ",)"); NEXT_TOKEN(vname)
if (*args == 0) { NEXT_TOKEN(type_params)
break;
}
if (*args == ')') {
args++;
continue;
}
name_length = args - name;
args = strip(args+1);
vname = args;
args = strpbrk(args, ",)");
if (*args == 0) {
break;
}
if (*args == ')') {
args++;
continue;
}
vname_length = args - vname;
args = strip(args+1);
type_params = args;
args = strpbrk(args, ",)");
if (*args == 0) {
break;
}
if (*args == ')') {
args++;
continue;
}
type_params_length = args - type_params;
args = strip(args+1);
type_params_length = MIN(type_params_length, sizeof(type_str)-1); type_params_length = MIN(type_params_length, sizeof(type_str)-1);
strncpy(type_str, type_params, type_params_length); strncpy(type_str, type_params, type_params_length);
type_str[type_params_length] = 0; type_str[type_params_length] = 0;
type = vcard_emul_type_from_string(type_str); type = vcard_emul_type_from_string(type_str);
type_params = args; NEXT_TOKEN(type_params)
args = strpbrk(args, ",)");
if (*args == 0) {
break;
}
if (*args == ')') {
args++;
continue;
}
type_params_length = args - type_params;
args = strip(args+1);
if (*args == 0) { if (*args == 0) {
break; break;