Subject: [PATCH] Remove uneccessary strncmp call in imap-send count_messages

From: Andy Parkins <[email-masked]>
Date: Sun, 22 Oct 2006 15:53:28 +0100

The strncmp() call in count_messages() was doing the same comparison that 
strstr() did, so there was no need to call it.  A little rewrite and 
count_messages() is simpler.

Obviously no huge benefit, as it's not frequently called.  I'm just getting 
familiar with git.

Signed-off-by: Andy Parkins <[email-masked]>
---
 imap-send.c |   13 ++++++-------
 1 files changed, 6 insertions(+), 7 deletions(-)

diff --git a/imap-send.c b/imap-send.c
index 16804ab..cfac17b 100644
--- a/imap-send.c
+++ b/imap-send.c
@@ -1198,15 +1198,14 @@ count_messages( msg_data_t *msg )
 	int count = 0;
 	char *p = msg->data;
 
-	while (1) {
-		if (!strncmp( "From ", p, 5 )) {
-			count++;
+	while (p) {
+		p = strstr( p, "From " );
+		if ( p != NULL ) {
+			if ( p == msg->data || *(p-1) == '\n' ) {
+				count++;
+			}
 			p += 5;
 		}
-		p = strstr( p+5, "\nFrom ");
-		if (!p)
-			break;
-		p++;
 	}
 	return count;
 }
-- 
1.4.2.3