From 8f3cdfb2de7b63533fe95e5a5c24374105842784 Mon Sep 17 00:00:00 2001
From: Tatsuhiro Tsujikawa <tatsuhiro.t@gmail.com>
Date: Thu, 20 May 2010 12:16:50 +0000
Subject: [PATCH] 2010-05-20  Tatsuhiro Tsujikawa 
 <t-tujikawa@users.sourceforge.net>

	Removed DownloadResult's ctor because it has many args.
	* src/DownloadResult.h
	* src/RequestGroup.cc
	* test/XmlRpcMethodTest.cc
---
 ChangeLog                |  7 +++++++
 src/DownloadResult.h     | 21 ---------------------
 src/RequestGroup.cc      | 24 ++++++++++++------------
 test/XmlRpcMethodTest.cc | 20 +++++++++-----------
 4 files changed, 28 insertions(+), 44 deletions(-)

diff --git a/ChangeLog b/ChangeLog
index 6931b279..448eaa43 100644
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,10 @@
+2010-05-20  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
+
+	Removed DownloadResult's ctor because it has many args.
+	* src/DownloadResult.h
+	* src/RequestGroup.cc
+	* test/XmlRpcMethodTest.cc
+
 2010-05-19  Tatsuhiro Tsujikawa  <t-tujikawa@users.sourceforge.net>
 
 	Code cleanup.
diff --git a/src/DownloadResult.h b/src/DownloadResult.h
index 54c0f924..76f15915 100644
--- a/src/DownloadResult.h
+++ b/src/DownloadResult.h
@@ -79,27 +79,6 @@ public:
   SharedHandle<Option> option;
 
   SharedHandle<MetadataInfo> metadataInfo;
-
-  DownloadResult(gid_t gid,
-                 const std::vector<SharedHandle<FileEntry> >& fileEntries,
-                 bool inMemoryDownload,
-                 uint64_t sessionDownloadLength,
-                 int64_t sessionTime,
-                 downloadresultcode::RESULT result,
-                 const std::vector<gid_t> followedBy,
-                 gid_t belongsTo,
-                 const SharedHandle<Option>& option,
-                 const SharedHandle<MetadataInfo>& metadataInfo):
-    gid(gid),
-    fileEntries(fileEntries),
-    inMemoryDownload(inMemoryDownload),
-    sessionDownloadLength(sessionDownloadLength),
-    sessionTime(sessionTime),
-    result(result),
-    followedBy(followedBy),
-    belongsTo(belongsTo),
-    option(option),
-    metadataInfo(metadataInfo) {}
 };
 
 typedef SharedHandle<DownloadResult> DownloadResultHandle;
diff --git a/src/RequestGroup.cc b/src/RequestGroup.cc
index 100afe2b..6bedbd8b 100644
--- a/src/RequestGroup.cc
+++ b/src/RequestGroup.cc
@@ -1057,18 +1057,18 @@ DownloadResultHandle RequestGroup::createDownloadResult() const
     sessionDownloadLength +=
       _segmentMan->calculateSessionDownloadLength();
   }
-  return
-    SharedHandle<DownloadResult>
-    (new DownloadResult(_gid,
-                        _downloadContext->getFileEntries(),
-                        _inMemoryDownload,
-                        sessionDownloadLength,
-                        _downloadContext->calculateSessionTime(),
-                        downloadResult(),
-                        _followedByGIDs,
-                        _belongsToGID,
-                        _option,
-                        _metadataInfo));
+  SharedHandle<DownloadResult> res(new DownloadResult());
+  res->gid = _gid;
+  res->fileEntries = _downloadContext->getFileEntries();
+  res->inMemoryDownload = _inMemoryDownload;
+  res->sessionDownloadLength = sessionDownloadLength;
+  res->sessionTime = _downloadContext->calculateSessionTime();
+  res->result = downloadResult();
+  res->followedBy = _followedByGIDs;
+  res->belongsTo = _belongsToGID;
+  res->option = _option;
+  res->metadataInfo = _metadataInfo;
+  return res;
 }
   
 void RequestGroup::reportDownloadFinished()
diff --git a/test/XmlRpcMethodTest.cc b/test/XmlRpcMethodTest.cc
index 77f88f06..cb049ab9 100644
--- a/test/XmlRpcMethodTest.cc
+++ b/test/XmlRpcMethodTest.cc
@@ -674,17 +674,15 @@ void XmlRpcMethodTest::testGatherStoppedDownload()
   std::vector<gid_t> followedBy;
   followedBy.push_back(3);
   followedBy.push_back(4);
-  SharedHandle<DownloadResult> d
-    (new DownloadResult(1,
-                        fileEntries,
-                        false,
-                        UINT64_MAX,
-                        1000,
-                        downloadresultcode::FINISHED,
-                        followedBy,
-                        2,
-                        SharedHandle<Option>(),
-                        SharedHandle<MetadataInfo>()));
+  SharedHandle<DownloadResult> d(new DownloadResult());
+  d->gid = 1;
+  d->fileEntries = fileEntries;
+  d->inMemoryDownload = false;
+  d->sessionDownloadLength = UINT64_MAX;
+  d->sessionTime = 1000;
+  d->result = downloadresultcode::FINISHED;
+  d->followedBy = followedBy;
+  d->belongsTo = 2;
   BDE entry = BDE::dict();
   gatherStoppedDownload(entry, d);