Use const reference in getData method

Plugins/org.commontk.dah.app/ctkDicomHostService.cpp

@@ -88,8 +88,8 @@ bool ctkDicomHostService::notifyDataAvailable(ctkDicomAppHosting::AvailableData
 
 //----------------------------------------------------------------------------
 QList<ctkDicomAppHosting::ObjectLocator> ctkDicomHostService::getData(
-  QList<QUuid> objectUUIDs, 
-  QList<QString> acceptableTransferSyntaxUIDs, 
+  const QList<QUuid>& objectUUIDs,
+  const QList<QString>& acceptableTransferSyntaxUIDs,
   bool includeBulkData)
 {
   return ctkDicomExchangeService::getData(objectUUIDs, acceptableTransferSyntaxUIDs, includeBulkData);

Plugins/org.commontk.dah.app/ctkDicomHostService_p.h

@@ -46,8 +46,8 @@ public:
   virtual bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData);
 
   virtual QList<ctkDicomAppHosting::ObjectLocator> getData(
-    QList<QUuid> objectUUIDs, 
-    QList<QString> acceptableTransferSyntaxUIDs, 
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
     bool includeBulkData);
 
   virtual void releaseData(QList<QUuid> objectUUIDs);

Plugins/org.commontk.dah.core/ctkDicomExchangeInterface.h

@@ -37,8 +37,8 @@ struct ctkDicomExchangeInterface
   virtual bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData) = 0;
 
   virtual QList<ctkDicomAppHosting::ObjectLocator> getData(
-    QList<QUuid> objectUUIDs, 
-    QList<QString> acceptableTransferSyntaxUIDs, 
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
     bool includeBulkData) = 0;
 
   virtual void releaseData(QList<QUuid> objectUUIDs) = 0;

Plugins/org.commontk.dah.core/ctkDicomExchangeService.cpp

@@ -51,8 +51,8 @@ bool ctkDicomExchangeService::notifyDataAvailable(
 
 //----------------------------------------------------------------------------
 QList<ctkDicomAppHosting::ObjectLocator> ctkDicomExchangeService::getData(
-    QList<QUuid> objectUUIDs,
-    QList<QString> acceptableTransferSyntaxUIDs, bool includeBulkData)
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs, bool includeBulkData)
 {
   //Q_D(ctkDicomService);
   QList<QtSoapType*> list;

Plugins/org.commontk.dah.core/ctkDicomExchangeService.h

@@ -38,8 +38,12 @@ public:
   virtual ~ctkDicomExchangeService();
 
   bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData);
-  QList<ctkDicomAppHosting::ObjectLocator> getData(QList<QUuid> objectUUIDs,
-                                             QList<QString> acceptableTransferSyntaxUIDs, bool includeBulkData);
+
+  QList<ctkDicomAppHosting::ObjectLocator> getData(
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
+    bool includeBulkData);
+
   void releaseData(QList<QUuid> objectUUIDs);
 
 };

Plugins/org.commontk.dah.exampleapp/ctkExampleDicomAppLogic.cpp

@@ -173,8 +173,8 @@ bool ctkExampleDicomAppLogic::notifyDataAvailable(ctkDicomAppHosting::AvailableD
 
 //----------------------------------------------------------------------------
 QList<ctkDicomAppHosting::ObjectLocator> ctkExampleDicomAppLogic::getData(
-  QList<QUuid> objectUUIDs,
-  QList<QString> acceptableTransferSyntaxUIDs,
+  const QList<QUuid>& objectUUIDs,
+  const QList<QString>& acceptableTransferSyntaxUIDs,
   bool includeBulkData)
 {
   Q_UNUSED(objectUUIDs)

Plugins/org.commontk.dah.exampleapp/ctkExampleDicomAppLogic_p.h

@@ -51,8 +51,8 @@ public:
   virtual bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData);
 
   virtual QList<ctkDicomAppHosting::ObjectLocator> getData(
-    QList<QUuid> objectUUIDs,
-    QList<QString> acceptableTransferSyntaxUIDs,
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
     bool includeBulkData);
 
   virtual void releaseData(QList<QUuid> objectUUIDs);

Plugins/org.commontk.dah.examplehost/ctkExampleDicomHost.cpp

@@ -117,8 +117,8 @@ bool ctkExampleDicomHost::notifyDataAvailable(ctkDicomAppHosting::AvailableData
 
 //----------------------------------------------------------------------------
 QList<ctkDicomAppHosting::ObjectLocator> ctkExampleDicomHost::getData(
-    QList<QUuid> objectUUIDs,
-    QList<QString> acceptableTransferSyntaxUIDs,
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
     bool includeBulkData)
 {
   Q_UNUSED(includeBulkData)

Plugins/org.commontk.dah.examplehost/ctkExampleDicomHost.h

@@ -50,8 +50,8 @@ public:
   // exchange methods
   virtual bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData);
   virtual QList<ctkDicomAppHosting::ObjectLocator> getData(
-      QList<QUuid> objectUUIDs,
-      QList<QString> acceptableTransferSyntaxUIDs,
+      const QList<QUuid>& objectUUIDs,
+      const QList<QString>& acceptableTransferSyntaxUIDs,
       bool includeBulkData);
   virtual void releaseData(QList<QUuid> objectUUIDs);
 

Plugins/org.commontk.dah.host/ctkDicomAppService.cpp

@@ -70,8 +70,8 @@ bool ctkDicomAppService::notifyDataAvailable(ctkDicomAppHosting::AvailableData d
 
 //----------------------------------------------------------------------------
 QList<ctkDicomAppHosting::ObjectLocator> ctkDicomAppService::getData(
-  QList<QUuid> objectUUIDs, 
-  QList<QString> acceptableTransferSyntaxUIDs, 
+  const QList<QUuid>& objectUUIDs,
+  const QList<QString>& acceptableTransferSyntaxUIDs,
   bool includeBulkData)
 {
   return ctkDicomExchangeService::getData(objectUUIDs, acceptableTransferSyntaxUIDs, includeBulkData);

Plugins/org.commontk.dah.host/ctkDicomAppService.h

@@ -41,8 +41,8 @@ public:
   virtual bool notifyDataAvailable(ctkDicomAppHosting::AvailableData data, bool lastData);
 
   virtual QList<ctkDicomAppHosting::ObjectLocator> getData(
-    QList<QUuid> objectUUIDs, 
-    QList<QString> acceptableTransferSyntaxUIDs, 
+    const QList<QUuid>& objectUUIDs,
+    const QList<QString>& acceptableTransferSyntaxUIDs,
     bool includeBulkData);
 
   virtual void releaseData(QList<QUuid> objectUUIDs);