2008-05-12 Tatsuhiro Tsujikawa <tujikawa at rednoah dot com>

	Changed method sinature:
	RequestGroup::getURIs
	* src/RequestGroup.cc
	* src/RequestGroup.h
	* test/Metalink2RequestGroupTest.cc

ChangeLog

@@ -1,3 +1,11 @@
+2008-05-12  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
+
+	Changed method sinature:
+	RequestGroup::getURIs
+	* src/RequestGroup.cc
+	* src/RequestGroup.h
+	* test/Metalink2RequestGroupTest.cc
+
 2008-05-12  Tatsuhiro Tsujikawa  <tujikawa at rednoah dot com>
 
 	Changed method signature:

src/RequestGroup.cc

@@ -754,11 +754,10 @@ void RequestGroup::initializePostDownloadHandler()
 #endif // ENABLE_METALINK
 }
 
-std::deque<std::string> RequestGroup::getUris() const
+void RequestGroup::getURIs(std::deque<std::string>& uris) const
 {
-  std::deque<std::string> temp(_spentUris.begin(), _spentUris.end());
-  temp.insert(temp.end(), _uris.begin(), _uris.end());
-  return temp;
+  uris.insert(uris.end(), _spentUris.begin(), _spentUris.end());
+  uris.insert(uris.end(), _uris.begin(), _uris.end());
 }
 
 bool RequestGroup::isDependencyResolved()
@@ -853,7 +852,8 @@ bool RequestGroup::needsFileAllocation() const
 
 DownloadResultHandle RequestGroup::createDownloadResult() const
 {
-  std::deque<std::string> uris = getUris();
+  std::deque<std::string> uris;
+  getURIs(uris);
   return
     SharedHandle<DownloadResult>(new DownloadResult(_gid,
 						    getFilePath(),

src/RequestGroup.h

@@ -179,7 +179,7 @@ public:
     return _spentUris;
   }
 
-  std::deque<std::string> getUris() const;
+  void getURIs(std::deque<std::string>& uris) const;
 
   /**
    * Compares expected filename with specified actualFilename.

test/Metalink2RequestGroupTest.cc

@@ -37,7 +37,8 @@ void Metalink2RequestGroupTest::testGenerate()
   // first file
   {
     SharedHandle<RequestGroup> rg = groups[0];
-    std::deque<std::string> uris = rg->getUris();
+    std::deque<std::string> uris;
+    rg->getURIs(uris);
     std::sort(uris.begin(), uris.end());
     CPPUNIT_ASSERT_EQUAL((size_t)2, uris.size());
     CPPUNIT_ASSERT_EQUAL(std::string("ftp://ftphost/aria2-0.5.2.tar.bz2"), uris[0]);
@@ -55,7 +56,8 @@ void Metalink2RequestGroupTest::testGenerate()
   // second file
   {
     SharedHandle<RequestGroup> rg = groups[1];
-    std::deque<std::string> uris = rg->getUris();
+    std::deque<std::string> uris;
+    rg->getURIs(uris);
     CPPUNIT_ASSERT_EQUAL((size_t)2, uris.size());
     SharedHandle<SingleFileDownloadContext> dctx
       (dynamic_pointer_cast<SingleFileDownloadContext>(rg->getDownloadContext()));
@@ -73,7 +75,8 @@ void Metalink2RequestGroupTest::testGenerate()
   // fifth file <- downloading .torrent file
   {
     SharedHandle<RequestGroup> rg = groups[4];
-    std::deque<std::string> uris = rg->getUris();
+    std::deque<std::string> uris;
+    rg->getURIs(uris);
     CPPUNIT_ASSERT_EQUAL((size_t)1, uris.size());
     CPPUNIT_ASSERT_EQUAL(std::string("http://host/torrent-http.integrated.torrent"),
 			 uris[0]);
@@ -90,7 +93,8 @@ void Metalink2RequestGroupTest::testGenerate()
 #else
     SharedHandle<RequestGroup> rg = groups[4];
 #endif // ENABLE_BITTORRENT
-    std::deque<std::string> uris = rg->getUris();
+    std::deque<std::string> uris;
+    rg->getURIs(uris);
     CPPUNIT_ASSERT_EQUAL((size_t)1, uris.size());
     CPPUNIT_ASSERT_EQUAL(std::string("http://host/torrent-http.integrated"), uris[0]);
     SharedHandle<SingleFileDownloadContext> dctx