ahp/busybox
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

libbb: make index_in_substrings return -1 on ambiguous matches

Browse source 
function                                             old     new   delta
index_in_substrings                                   67      93     +26

Signed-off-by: Denys Vlasenko <vda.linux@googlemail.com>
This commit is contained in:
Denys Vlasenko 2010-04-03 00:52:16 +02:00
parent 05d1a32192
commit 8a659f6ff9
3 changed files with 24 additions and 7 deletions
Download patch file Download diff file Expand all files Collapse all files

23
libbb/compare_string_array.c
View file

@ -53,19 +53,24 @@ int FAST_FUNC index_in_substr_array(const char *const string_array[], const char
int FAST_FUNC index_in_substrings(const char *strings, const char *key)
{
int len = strlen(key);
int matched_idx = -1;
const int len = strlen(key);
if (len) {
int idx = 0;
while (*strings) {
if (strncmp(strings, key, len) == 0) {
return idx;
if (strings[len] == '\0')
return idx; /* exact match */
if (matched_idx >= 0)
return -1; /* ambiguous match */
matched_idx = idx;
}
strings += strlen(strings) + 1; /* skip NUL */
idx++;
}
}
return -1;
return matched_idx;
}
const char* FAST_FUNC nth_string(const char *strings, int n)
@ -76,3 +81,15 @@ const char* FAST_FUNC nth_string(const char *strings, int n)
}
return strings;
}
#ifdef UNUSED_SO_FAR /* only brctl.c needs it yet */
/* Returns 0 for no, 1 for yes or a negative value on error. */
smallint FAST_FUNC yesno(const char *str)
{
static const char no_yes[] ALIGN1 =
"0\0" "off\0" "no\0"
"1\0" "on\0" "yes\0";
int ret = index_in_substrings(no_yes, str);
return ret / 3;
}
#endif