diff options
| author | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2010-10-30 16:02:15 +0000 |
|---|---|---|
| committer | Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com> | 2010-10-30 16:02:15 +0000 |
| commit | 236e64cb25a054eed3fa8888168e71d09decf561 (patch) | |
| tree | 230c86632d06252af05a1803b63417f7873033f3 /src/HttpResponseCommand.cc | |
| parent | 6ecfa4aad1d49b14039aec1004539019462ec095 (diff) | |
2010-10-30 Tatsuhiro Tsujikawa <t-tujikawa@users.sourceforge.net>
Use unnamed namespace instead of static keyword.
* src/AbstractCommand.cc
* src/AdaptiveURISelector.cc
* src/Base64.cc
* src/BitfieldMan.cc
* src/BtDependency.cc
* src/ConsoleStatCalc.cc
* src/ContentTypeRequestGroupCriteria.cc
* src/CookieStorage.cc
* src/DHTMessageFactoryImpl.cc
* src/DHTRoutingTableDeserializer.cc
* src/DefaultBtAnnounce.cc
* src/DefaultBtProgressInfoFile.cc
* src/DefaultPeerStorage.cc
* src/DefaultPieceStorage.cc
* src/DownloadCommand.cc
* src/DownloadEngine.cc
* src/EpollEventPoll.cc
* src/ExpatMetalinkProcessor.cc
* src/ExpatXmlRpcRequestProcessor.cc
* src/FileEntry.cc
* src/HttpRequest.cc
* src/HttpRequestCommand.cc
* src/HttpResponseCommand.cc
* src/KqueueEventPoll.cc
* src/LongestSequencePieceSelector.cc
* src/MetalinkParserStateV3Impl.cc
* src/MetalinkParserStateV4Impl.cc
* src/MultiDiskAdaptor.cc
* src/MultiUrlRequestInfo.cc
* src/OptionParser.cc
* src/PeerSessionResource.cc
* src/PortEventPoll.cc
* src/Request.cc
* src/RequestGroupMan.cc
* src/SelectEventPoll.cc
* src/SessionSerializer.cc
* src/SimpleLogFormatter.cc
* src/Sqlite3CookieParser.cc
* src/TrackerWatcherCommand.cc
* src/XML2SAXMetalinkProcessor.cc
* src/Xml2XmlRpcRequestProcessor.cc
* src/XmlRpcMethod.cc
* src/XmlRpcMethodImpl.cc
* src/XmlRpcResponse.cc
* src/base32.cc
* src/bencode2.cc
* src/bittorrent_helper.cc
* src/download_helper.cc
* src/main.cc
* src/messageDigest.cc
* src/option_processing.cc
* src/util.cc
* test/AnnounceListTest.cc
* test/BtRegistryTest.cc
* test/DHTBucketTest.cc
* test/DHTRoutingTableTest.cc
* test/DefaultBtAnnounceTest.cc
* test/FileEntryTest.cc
* test/FtpConnectionTest.cc
* test/MSEHandshakeTest.cc
* test/MagnetTest.cc
* test/XmlRpcMethodTest.cc
* test/array_funTest.cc
Diffstat (limited to 'src/HttpResponseCommand.cc')
| -rw-r--r-- | src/HttpResponseCommand.cc | 98 |
1 files changed, 48 insertions, 50 deletions
diff --git a/src/HttpResponseCommand.cc b/src/HttpResponseCommand.cc index a0fa41a9..2971a0dc 100644 --- a/src/HttpResponseCommand.cc +++ b/src/HttpResponseCommand.cc @@ -79,13 +79,54 @@ namespace aria2 { -static SharedHandle<StreamFilter> getTransferEncodingStreamFilter +namespace { +SharedHandle<StreamFilter> getTransferEncodingStreamFilter (const SharedHandle<HttpResponse>& httpResponse, - const SharedHandle<StreamFilter>& delegate = SharedHandle<StreamFilter>()); + const SharedHandle<StreamFilter>& delegate = SharedHandle<StreamFilter>()) +{ + SharedHandle<StreamFilter> filter; + if(httpResponse->isTransferEncodingSpecified()) { + filter = httpResponse->getTransferEncodingStreamFilter(); + if(filter.isNull()) { + throw DL_ABORT_EX + (StringFormat(EX_TRANSFER_ENCODING_NOT_SUPPORTED, + httpResponse->getTransferEncoding().c_str()).str()); + } + filter->init(); + filter->installDelegate(delegate); + } + if(filter.isNull()) { + filter = delegate; + } + return filter; +} +} // namespace -static SharedHandle<StreamFilter> getContentEncodingStreamFilter +namespace { +SharedHandle<StreamFilter> getContentEncodingStreamFilter (const SharedHandle<HttpResponse>& httpResponse, - const SharedHandle<StreamFilter>& delegate = SharedHandle<StreamFilter>()); + const SharedHandle<StreamFilter>& delegate = SharedHandle<StreamFilter>()) +{ + SharedHandle<StreamFilter> filter; + if(httpResponse->isContentEncodingSpecified()) { + filter = httpResponse->getContentEncodingStreamFilter(); + if(filter.isNull()) { + LogFactory::getInstance()->info + ("Content-Encoding %s is specified, but the current implementation" + "doesn't support it. The decoding process is skipped and the" + "downloaded content will be still encoded.", + httpResponse->getContentEncoding().c_str()); + } else { + filter->init(); + filter->installDelegate(delegate); + } + } + if(filter.isNull()) { + filter = delegate; + } + return filter; +} +} // namespace HttpResponseCommand::HttpResponseCommand (cuid_t cuid, @@ -305,51 +346,6 @@ bool HttpResponseCommand::handleDefaultEncoding return true; } -static SharedHandle<StreamFilter> getTransferEncodingStreamFilter -(const SharedHandle<HttpResponse>& httpResponse, - const SharedHandle<StreamFilter>& delegate) -{ - SharedHandle<StreamFilter> filter; - if(httpResponse->isTransferEncodingSpecified()) { - filter = httpResponse->getTransferEncodingStreamFilter(); - if(filter.isNull()) { - throw DL_ABORT_EX - (StringFormat(EX_TRANSFER_ENCODING_NOT_SUPPORTED, - httpResponse->getTransferEncoding().c_str()).str()); - } - filter->init(); - filter->installDelegate(delegate); - } - if(filter.isNull()) { - filter = delegate; - } - return filter; -} - -static SharedHandle<StreamFilter> getContentEncodingStreamFilter -(const SharedHandle<HttpResponse>& httpResponse, - const SharedHandle<StreamFilter>& delegate) -{ - SharedHandle<StreamFilter> filter; - if(httpResponse->isContentEncodingSpecified()) { - filter = httpResponse->getContentEncodingStreamFilter(); - if(filter.isNull()) { - LogFactory::getInstance()->info - ("Content-Encoding %s is specified, but the current implementation" - "doesn't support it. The decoding process is skipped and the" - "downloaded content will be still encoded.", - httpResponse->getContentEncoding().c_str()); - } else { - filter->init(); - filter->installDelegate(delegate); - } - } - if(filter.isNull()) { - filter = delegate; - } - return filter; -} - bool HttpResponseCommand::handleOtherEncoding (const SharedHandle<HttpResponse>& httpResponse) { // We assume that RequestGroup::getTotalLength() == 0 here @@ -448,7 +444,8 @@ bool HttpResponseCommand::skipResponseBody return true; } -static bool decideFileAllocation +namespace { +bool decideFileAllocation (const SharedHandle<StreamFilter>& filter) { #ifdef HAVE_LIBZ @@ -463,6 +460,7 @@ static bool decideFileAllocation #endif // HAVE_LIBZ return true; } +} // namespace HttpDownloadCommand* HttpResponseCommand::createHttpDownloadCommand (const SharedHandle<HttpResponse>& httpResponse, |
