From fd0ce912f6515d812d68adb9e56be63fda38efed Mon Sep 17 00:00:00 2001
From: "jst%mozilla.jstenback.com"
 <jst%mozilla.jstenback.com@18797224-902f-48f8-a5cc-f745e15eee43>
Date: Tue, 13 Apr 2004 03:53:38 +0000
Subject: [PATCH] Fixing bug 227497. Cleaning up some string usage in
 nsLDAPChannel. r+sr=peterv@propagandism.org

git-svn-id: svn://10.0.0.236/trunk@154767 18797224-902f-48f8-a5cc-f745e15eee43
---
 .../directory/xpcom/base/src/nsLDAPChannel.cpp | 18 ++++++++----------
 1 file changed, 8 insertions(+), 10 deletions(-)

diff --git a/mozilla/directory/xpcom/base/src/nsLDAPChannel.cpp b/mozilla/directory/xpcom/base/src/nsLDAPChannel.cpp
index 4908a74a49a..4a4a8b377bf 100644
--- a/mozilla/directory/xpcom/base/src/nsLDAPChannel.cpp
+++ b/mozilla/directory/xpcom/base/src/nsLDAPChannel.cpp
@@ -897,7 +897,7 @@ nsLDAPChannel::OnLDAPSearchEntry(nsILDAPMessage *aMessage)
 {
     nsresult rv;
     nsCAutoString dn;
-    nsString entry;
+    nsCString entry;
 
     PR_LOG(gLDAPLogModule, PR_LOG_DEBUG, ("entry returned!\n"));
 
@@ -908,8 +908,7 @@ nsLDAPChannel::OnLDAPSearchEntry(nsILDAPMessage *aMessage)
     NS_ENSURE_SUCCESS(rv, rv);
 
     entry.SetCapacity(256);
-    entry = NS_LITERAL_STRING("dn: ") + NS_ConvertUTF8toUCS2(dn)
-        + NS_LITERAL_STRING("\n");
+    entry = NS_LITERAL_CSTRING("dn: ") + dn + NS_LITERAL_CSTRING("\n");
 
     char **attrs;
     PRUint32 attrCount;
@@ -947,10 +946,10 @@ nsLDAPChannel::OnLDAPSearchEntry(nsILDAPMessage *aMessage)
         // print all values of this attribute
         //
         for ( PRUint32 j=0 ; j < valueCount; j++ ) {
-            AppendASCIItoUTF16(attrs[i], entry);
-            entry.Append(NS_LITERAL_STRING(": "));
-            entry.Append(vals[j]);
-            entry.Append(NS_LITERAL_STRING("\n"));
+            entry.Append(attrs[i]);
+            entry.Append(": ");
+            AppendUTF16toUTF8(vals[j], entry);
+            entry.Append('\n');
         }
         NS_FREE_XPCOM_ALLOCATED_POINTER_ARRAY(valueCount, vals);
 
@@ -967,7 +966,7 @@ nsLDAPChannel::OnLDAPSearchEntry(nsILDAPMessage *aMessage)
 
     // separate this entry from the next
     //
-    entry.Append(NS_LITERAL_STRING("\n"));
+    entry.Append('\n');
 
     // do the write
     // XXX better err handling
@@ -975,8 +974,7 @@ nsLDAPChannel::OnLDAPSearchEntry(nsILDAPMessage *aMessage)
     PRUint32 bytesWritten = 0;
     PRUint32 entryLength = entry.Length();
 
-    rv = mReadPipeOut->Write(NS_ConvertUCS2toUTF8(entry).get(),
-                             entryLength, &bytesWritten);
+    rv = mReadPipeOut->Write(entry.get(), entryLength, &bytesWritten);
     NS_ENSURE_SUCCESS(rv, rv);
 
     // short writes shouldn't happen on blocking pipes!