Permalink
Browse files

Remove useless tests.

Turn
   if (*x && apr_isspace(*x))
into
   if (apr_isspace(*x))

git-svn-id: https://svn.apache.org/repos/asf/httpd/httpd/trunk@1452128 13f79535-47bb-0310-9956-ffa450edef68
  • Loading branch information...
1 parent bb79f2f commit d6f89a995a1c9e80bc7771e76b413d12d403ddf8 Christophe Jaillet committed Mar 3, 2013
@@ -841,7 +841,7 @@ static apr_status_t read_table(cache_handle_t *handle, request_rec *r,
}
*l++ = '\0';
- while (*l && apr_isspace(*l)) {
+ while (apr_isspace(*l)) {
++l;
}
@@ -670,7 +670,7 @@ static meta *metafix(request_rec *r, const char *buf)
if (p != NULL) {
while (*p) {
p += 7;
- while (*p && apr_isspace(*p))
+ while (apr_isspace(*p))
++p;
if (*p != '=')
continue;
@@ -686,7 +686,7 @@ static int imap_handler_internal(request_rec *r)
if (!*string_pos) { /* need at least two fields */
goto need_2_fields;
}
- while(*string_pos && apr_isspace(*string_pos)) { /* past whitespace */
+ while (apr_isspace(*string_pos)) { /* past whitespace */
++string_pos;
}
@@ -366,7 +366,7 @@ static float atoq(const char *string)
return 1.0f;
}
- while (*string && apr_isspace(*string)) {
+ while (apr_isspace(*string)) {
++string;
}
@@ -464,7 +464,7 @@ static const char *get_entry(apr_pool_t *p, accept_rec *result,
}
*cp++ = '\0'; /* Delimit var */
- while (*cp && (apr_isspace(*cp) || *cp == '=')) {
+ while (apr_isspace(*cp) || *cp == '=') {
++cp;
}
@@ -757,7 +757,7 @@ static enum header_state get_header_line(char *buffer, int len, apr_file_t *map)
/* If blank, just return it --- this ends information on this variant */
- for (cp = buffer; (*cp && apr_isspace(*cp)); ++cp) {
+ for (cp = buffer; apr_isspace(*cp); ++cp) {
continue;
}
@@ -924,7 +924,7 @@ static char *lcase_header_name_return_body(char *header, request_rec *r)
do {
++cp;
- } while (*cp && apr_isspace(*cp));
+ } while (apr_isspace(*cp));
if (!*cp) {
ap_log_rerror(APLOG_MARK, APLOG_ERR, 0, r, APLOGNO(00682)
@@ -1255,7 +1255,7 @@ static char *lookup_map_txtfile(request_rec *r, const char *file, char *key)
}
/* jump to the value */
- while (*p && apr_isspace(*p)) {
+ while (apr_isspace(*p)) {
++p;
}
@@ -237,7 +237,7 @@ static int scan_meta_file(request_rec *r, apr_file_t *f)
}
*l++ = '\0';
- while (*l && apr_isspace(*l))
+ while (apr_isspace(*l))
++l;
if (!strcasecmp(w, "Content-type")) {
@@ -722,7 +722,7 @@ static int do_headers_fixup(request_rec *r, apr_table_t *headers,
while (*val) {
const char *tok_start;
- while (*val && apr_isspace(*val))
+ while (apr_isspace(*val))
++val;
tok_start = val;
View
@@ -784,7 +784,7 @@ AP_DECLARE(char *) ap_getword_conf(apr_pool_t *p, const char **line)
char *res;
char quote;
- while (*str && apr_isspace(*str))
+ while (apr_isspace(*str))
++str;
if (!*str) {
@@ -816,7 +816,7 @@ AP_DECLARE(char *) ap_getword_conf(apr_pool_t *p, const char **line)
res = substring_conf(p, str, strend - str, 0);
}
- while (*strend && apr_isspace(*strend))
+ while (apr_isspace(*strend))
++strend;
*line = strend;
return res;
@@ -1406,7 +1406,7 @@ AP_DECLARE(char *) ap_get_token(apr_pool_t *p, const char **accept_line,
/* Find first non-white byte */
- while (*ptr && apr_isspace(*ptr))
+ while (apr_isspace(*ptr))
++ptr;
tok_start = ptr;
@@ -1428,7 +1428,7 @@ AP_DECLARE(char *) ap_get_token(apr_pool_t *p, const char **accept_line,
/* Advance accept_line pointer to the next non-white byte */
- while (*ptr && apr_isspace(*ptr))
+ while (apr_isspace(*ptr))
++ptr;
*accept_line = ptr;
View
@@ -566,7 +566,7 @@ AP_DECLARE(int) ap_scan_script_header_err_core_ex(request_rec *r, char *buffer,
}
*l++ = '\0';
- while (*l && apr_isspace(*l)) {
+ while (apr_isspace(*l)) {
++l;
}
View
@@ -138,7 +138,7 @@ static apr_status_t to_dbm(apr_dbm_t *dbm, apr_file_t *fp, apr_pool_t *pool)
dbmkey.dptr = apr_pstrmemdup(p, line, c - line);
dbmkey.dsize = (c - line);
- while (*c && apr_isspace(*c)) {
+ while (apr_isspace(*c)) {
++c;
}

0 comments on commit d6f89a9

Please sign in to comment.