[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-commits
Subject:    KDE/kdelibs/kioslave/http
From:       Dawit Alemayehu <adawit () kde ! org>
Date:       2007-09-01 16:55:01
Message-ID: 1188665701.515148.21312.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 707367 by adawit:

- Use QString instead of QStringList to store HTTP response headers. This
  should fix the kio_http portion of missing headers reported by Maksim.


 M  +3 -3      http.cpp  
 M  +1 -1      http.h  


--- trunk/KDE/kdelibs/kioslave/http/http.cpp #707366:707367
@@ -2586,7 +2586,7 @@
   // Send the response header if it was requested
   if ( config()->readEntry("PropagateHttpHeader", false) )
   {
-    setMetaData("HTTP-Headers", m_responseHeader.join("\n"));
+    setMetaData("HTTP-Headers", m_responseHeader);
     sendMetaData();
   }
   m_responseHeader.clear();
@@ -2606,7 +2606,7 @@
   // Check
   if (m_request.bCachedRead)
   {
-     m_responseHeader << "HTTP-CACHE";
+     m_responseHeader = QString::fromLatin1("HTTP-CACHE");
      // Read header from cache...
      char buffer[4097];
      if (!gzgets(m_request.fcache, buffer, 4096) )
@@ -2777,7 +2777,7 @@
 
     // Store the the headers so they can be passed to the
     // calling application later
-    m_responseHeader << QString::fromLatin1(buf);
+    m_responseHeader += QString::fromLatin1(buf);
 
     if ((strncasecmp(buf, "HTTP", 4) == 0) ||
         (strncasecmp(buf, "ICY ", 4) == 0)) // Shoutcast support
--- trunk/KDE/kdelibs/kioslave/http/http.h #707366:707367
@@ -497,7 +497,7 @@
 
 //--- Settings related to a single response only
   bool m_bRedirect; // Indicates current request is a redirection
-  QStringList m_responseHeader; // All headers
+  QString m_responseHeader; // All response headers
 
 
   // Language/Encoding related
[prev in list] [next in list] [prev in thread] [next in thread]