Fix #106: use stock strlen() & strcmp().

The binary size increases from 49184 to 49216.
This commit is contained in:
Arun Prakash Jana 2018-05-09 16:57:01 +05:30
parent 3b02014897
commit 5902af230d
No known key found for this signature in database
GPG key ID: A75979F35C080412

66
nnn.c
View file

@ -443,24 +443,6 @@ xrealloc(void *pcur, size_t len)
return pmem; return pmem;
} }
/*
* Custom xstrlen()
*/
static size_t
xstrlen(const char *s)
{
static size_t len;
len = 0;
if (!s)
return len;
while (*s)
++len, ++s;
return len;
}
/* /*
* Just a safe strncpy(3) * Just a safe strncpy(3)
* Always null ('\0') terminates if both src and dest are valid pointers. * Always null ('\0') terminates if both src and dest are valid pointers.
@ -477,7 +459,7 @@ xstrlcpy(char *dest, const char *src, size_t n)
if (!src || !dest || !n) if (!src || !dest || !n)
return 0; return 0;
len = xstrlen(src) + 1; len = strlen(src) + 1;
if (n > len) if (n > len)
n = len; n = len;
else if (len > n) else if (len > n)
@ -519,22 +501,6 @@ xstrlcpy(char *dest, const char *src, size_t n)
return len; return len;
} }
/*
* Custom strcmp(), just what we need.
* Returns 0 if same, -ve if s1 < s2, +ve if s1 > s2.
*/
static int
xstrcmp(const char *s1, const char *s2)
{
if (!s1 || !s2)
return -1;
while (*s1 && *s1 == *s2)
++s1, ++s2;
return *s1 - *s2;
}
/* /*
* The poor man's implementation of memrchr(3). * The poor man's implementation of memrchr(3).
* We are only looking for '/' in this program. * We are only looking for '/' in this program.
@ -575,7 +541,7 @@ xdirname(const char *path)
xstrlcpy(buf, path, PATH_MAX); xstrlcpy(buf, path, PATH_MAX);
/* Find last '/'. */ /* Find last '/'. */
last_slash = xmemrchr((uchar *)buf, '/', xstrlen(buf)); last_slash = xmemrchr((uchar *)buf, '/', strlen(buf));
if (last_slash != NULL && last_slash != buf && last_slash[1] == '\0') { if (last_slash != NULL && last_slash != buf && last_slash[1] == '\0') {
/* Determine whether all remaining characters are slashes. */ /* Determine whether all remaining characters are slashes. */
@ -627,7 +593,7 @@ xbasename(char *path)
{ {
static char *base; static char *base;
base = xmemrchr((uchar *)path, '/', xstrlen(path)); base = xmemrchr((uchar *)path, '/', strlen(path));
return base ? base + 1 : path; return base ? base + 1 : path;
} }
@ -888,7 +854,7 @@ strstrip(char *s)
if (!s || !*s) if (!s || !*s)
return s; return s;
size_t len = xstrlen(s) - 1; size_t len = strlen(s) - 1;
while (len != 0 && (isspace(s[len]) || s[len] == '/')) while (len != 0 && (isspace(s[len]) || s[len] == '/'))
--len; --len;
@ -1407,7 +1373,7 @@ get_bm_loc(char *key, char *buf)
return NULL; return NULL;
for (r = 0; bookmark[r].key && r < BM_MAX; ++r) { for (r = 0; bookmark[r].key && r < BM_MAX; ++r) {
if (xstrcmp(bookmark[r].key, key) == 0) { if (strcmp(bookmark[r].key, key) == 0) {
if (bookmark[r].loc[0] == '~') { if (bookmark[r].loc[0] == '~') {
char *home = getenv("HOME"); char *home = getenv("HOME");
@ -2239,7 +2205,7 @@ dentfind(struct entry *dents, const char *fname, int n)
DPRINTF_S(fname); DPRINTF_S(fname);
for (i = 0; i < n; ++i) for (i = 0; i < n; ++i)
if (xstrcmp(fname, dents[i].name) == 0) if (strcmp(fname, dents[i].name) == 0)
return i; return i;
return 0; return 0;
@ -2314,7 +2280,7 @@ redraw(char *path)
} }
/* Strip trailing slashes */ /* Strip trailing slashes */
for (i = xstrlen(path) - 1; i > 0; --i) for (i = strlen(path) - 1; i > 0; --i)
if (path[i] == '/') if (path[i] == '/')
path[i] = '\0'; path[i] = '\0';
else else
@ -2333,7 +2299,7 @@ redraw(char *path)
ncols = PATH_MAX; ncols = PATH_MAX;
/* No text wrapping in cwd line */ /* No text wrapping in cwd line */
/* Show CWD: - xstrlen(CWD) - 1 = 6 */ /* Show CWD: - strlen(CWD) - 1 = 6 */
g_buf[ncols - 6] = '\0'; g_buf[ncols - 6] = '\0';
printw(CWD "%s\n\n", g_buf); printw(CWD "%s\n\n", g_buf);
@ -2732,7 +2698,7 @@ nochange:
if (truecd == 0) { if (truecd == 0) {
/* Probable change in dir */ /* Probable change in dir */
/* No-op if it's the same directory */ /* No-op if it's the same directory */
if (xstrcmp(path, newpath) == 0) if (strcmp(path, newpath) == 0)
break; break;
oldname[0] = '\0'; oldname[0] = '\0';
@ -2768,7 +2734,7 @@ nochange:
goto nochange; goto nochange;
} }
if (xstrcmp(path, dir) == 0) { if (strcmp(path, dir) == 0) {
break; break;
} }
@ -2787,7 +2753,7 @@ nochange:
case SEL_CDLAST: // fallthrough case SEL_CDLAST: // fallthrough
case SEL_VISIT: case SEL_VISIT:
if (sel == SEL_VISIT) { if (sel == SEL_VISIT) {
if (xstrcmp(mark, path) == 0) if (strcmp(mark, path) == 0)
break; break;
tmp = mark; tmp = mark;
@ -2832,7 +2798,7 @@ nochange:
if (!xdiraccess(newpath)) if (!xdiraccess(newpath))
goto nochange; goto nochange;
if (xstrcmp(path, newpath) == 0) if (strcmp(path, newpath) == 0)
break; break;
oldname[0] = '\0'; oldname[0] = '\0';
@ -3087,7 +3053,7 @@ nochange:
break; break;
/* Allow only relative, same dir paths */ /* Allow only relative, same dir paths */
if (tmp[0] == '/' || xstrcmp(xbasename(tmp), tmp) != 0) { if (tmp[0] == '/' || strcmp(xbasename(tmp), tmp) != 0) {
printmsg(messages[STR_INPUT_ID]); printmsg(messages[STR_INPUT_ID]);
goto nochange; goto nochange;
} }
@ -3171,13 +3137,13 @@ nochange:
break; break;
/* Allow only relative, same dir paths */ /* Allow only relative, same dir paths */
if (tmp[0] == '/' || xstrcmp(xbasename(tmp), tmp) != 0) { if (tmp[0] == '/' || strcmp(xbasename(tmp), tmp) != 0) {
printmsg(messages[STR_INPUT_ID]); printmsg(messages[STR_INPUT_ID]);
goto nochange; goto nochange;
} }
/* Skip renaming to same name */ /* Skip renaming to same name */
if (xstrcmp(tmp, dents[cur].name) == 0) if (strcmp(tmp, dents[cur].name) == 0)
break; break;
/* Open the descriptor to currently open directory */ /* Open the descriptor to currently open directory */
@ -3253,7 +3219,7 @@ nochange:
goto begin; goto begin;
case SEL_RUNARG: case SEL_RUNARG:
run = xgetenv(env, run); run = xgetenv(env, run);
if ((!run || !run[0]) && (xstrcmp("VISUAL", env) == 0)) if ((!run || !run[0]) && (strcmp("VISUAL", env) == 0))
run = editor ? editor : xgetenv("EDITOR", "vi"); run = editor ? editor : xgetenv("EDITOR", "vi");
spawn(run, dents[cur].name, NULL, path, F_NORMAL); spawn(run, dents[cur].name, NULL, path, F_NORMAL);
break; break;