From: Dave West <davew@uncensored.citadel.org>
Date: Thu, 23 Aug 2007 09:52:33 +0000 (+0000)
Subject: Removed IsEmptyStr from some loops as its quicker this way (probably).
X-Git-Tag: v7.86~3135
X-Git-Url: https://code.citadel.org/?p=citadel.git;a=commitdiff_plain;h=ce2ddd8556ca44adfb511ac4a39a34753023141f

Removed IsEmptyStr from some loops as its quicker this way (probably).
---

diff --git a/citadel/modules/imap/imap_fetch.c b/citadel/modules/imap/imap_fetch.c
index 9f08131df..49818e438 100644
--- a/citadel/modules/imap/imap_fetch.c
+++ b/citadel/modules/imap/imap_fetch.c
@@ -510,11 +510,11 @@ void imap_strip_headers(char *section) {
 	if (!strncasecmp(which_fields, "HEADER.FIELDS.NOT", 17))
 		headers_not = 1;
 
-	for (i=0; !IsEmptyStr(&which_fields[i]); ++i) {
+	for (i=0; which_fields[i]; ++i) {
 		if (which_fields[i]=='(')
 			strcpy(which_fields, &which_fields[i+1]);
 	}
-	for (i=0; !IsEmptyStr(&which_fields[i]); ++i) {
+	for (i=0; which_fields[i]; ++i) {
 		if (which_fields[i]==')') {
 			which_fields[i] = 0;
 			break;
@@ -1096,7 +1096,7 @@ void imap_handle_macros(char *str) {
 	int i;
 	int nest = 0;
 
-	for (i=0; !IsEmptyStr(&str[i]); ++i) {
+	for (i=0; str[i]; ++i) {
 		if (str[i]=='(') ++nest;
 		if (str[i]=='[') ++nest;
 		if (str[i]=='<') ++nest;
@@ -1142,7 +1142,7 @@ int imap_extract_data_items(char **argv, char *items) {
 	long initial_len;
 
 	/* Convert all whitespace to ordinary space characters. */
-	for (i=0; !IsEmptyStr(&items[i]); ++i) {
+	for (i=0; items[i]; ++i) {
 		if (isspace(items[i])) items[i]=' ';
 	}