zorba-coders team mailing list archive
-
zorba-coders team
-
Mailing list archive
-
Message #10172
[Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
Paul J. Lucas has proposed merging lp:~paul-lucas/zorba/bug-1002867 into lp:zorba.
Requested reviews:
David Graf (davidagraf)
Related bugs:
Bug #1002867 in Zorba: "resulting base64 in http-client is wrong"
https://bugs.launchpad.net/zorba/+bug/1002867
For more details, see:
https://code.launchpad.net/~paul-lucas/zorba/bug-1002867/+merge/106855
No longer setting the charset of an overridden media type to a default value.
--
https://code.launchpad.net/~paul-lucas/zorba/bug-1002867/+merge/106855
Your team Zorba Coders is subscribed to branch lp:zorba.
=== modified file 'modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp'
--- modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp 2012-05-10 04:12:09 +0000
+++ modules/com/zorba-xquery/www/modules/http-client.xq.src/http_response_parser.cpp 2012-05-22 16:46:23 +0000
@@ -41,8 +41,7 @@
std::string::size_type pos = s.find( ';' );
*mime_type = s.substr( 0, pos );
- // The HTTP/1.1 spec says that the default charset is ISO-8859-1.
- *charset = "ISO-8859-1";
+ charset->clear();
if ( pos != std::string::npos ) {
//
@@ -109,7 +108,8 @@
std::auto_ptr<std::istream> lStream;
try {
- if ( transcode::is_necessary( theCurrentCharset.c_str() ) ) {
+ if ( !theCurrentCharset.empty() &&
+ transcode::is_necessary( theCurrentCharset.c_str() ) ) {
lStream.reset(
new transcode::stream<std::istream>(
theCurrentCharset.c_str(), theStreamBuffer
Follow ups
-
[Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: noreply, 2012-05-23
-
[Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Zorba Build Bot, 2012-05-23
-
[Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Zorba Build Bot, 2012-05-23
-
[Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Paul J. Lucas, 2012-05-23
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: David Graf, 2012-05-23
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Matthias Brantner, 2012-05-22
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Paul J. Lucas, 2012-05-22
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Matthias Brantner, 2012-05-22
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Paul J. Lucas, 2012-05-22
-
Re: [Merge] lp:~paul-lucas/zorba/bug-1002867 into lp:zorba
From: Matthias Brantner, 2012-05-22