backing out a jag's change. mLength on returnString is not correct, so doing returnString.Length()

cause problems, but strlen() works.  bienvenu's looking into why our string foo is doing that to mLength,
but this gets the tree open. sr=bienvenu

mailnews/imap/src/nsIMAPGenericParser.cpp

@@ -491,7 +491,7 @@ char *nsIMAPGenericParser::CreateQuoted(PRBool /*skipToEnd*/)
 //			fCurrentTokenPlaceHolder -= charDiff;
 //			if (!nsCRT::strcmp(fCurrentTokenPlaceHolder, CRLF))
 //				fAtEndOfLine = PR_TRUE;
-			AdvanceTokenizerStartingPoint ((fNextToken - fLineOfTokens) + returnString.Length() + 2);
+			AdvanceTokenizerStartingPoint ((fNextToken - fLineOfTokens) + nsCRT::strlen(returnString) + 2);
 			if (!nsCRT::strcmp(fLineOfTokens, CRLF))
 				fAtEndOfLine = PR_TRUE;
 		}