better implementation of setCapacity

pvDataApp/factory/PVDataCreateFactory.cpp

@@ -223,15 +223,17 @@ void DefaultPVArray<T>::setCapacity(size_t capacity)
     }
     size_t length = PVArray::getLength();
     if(length>capacity) length = capacity;
-    /*
-    std::vector<T> array;
-    array.reserve(capacity);
-    array.resize(length);
-    T * from = get();
-    for (size_t i=0; i<length; i++) array[i] = from[i];
-    value->swap(array);
-    */
-    value->reserve(capacity);
+    size_t oldCapacity = PVArray::getCapacity();
+    if(oldCapacity>capacity) {
+        std::vector<T> array;
+        array.reserve(capacity);
+        array.resize(length);
+        T * from = get();
+        for (size_t i=0; i<length; i++) array[i] = from[i];
+        value->swap(array);
+    } else {
+        value->reserve(capacity);
+    }
     PVArray::setCapacityLength(capacity,length);
 }
 

pvDataApp/factory/PVStructureArray.cpp

@@ -87,14 +87,19 @@ void PVStructureArray::setCapacity(size_t capacity) {
         PVField::message(message, errorMessage);
         return;
     }
-    size_t length = getCapacity();
+    size_t length = getLength();
     if(length>capacity) length = capacity;
-    PVStructurePtrArray array;
-    array.reserve(capacity);
-    array.resize(length);
-    PVStructurePtr * from = get();
-    for (size_t i=0; i<length; i++) array[i] = from[i];
-    value->swap(array);
+    size_t oldCapacity = getCapacity();
+    if(oldCapacity>capacity) {
+        PVStructurePtrArray array;
+        array.reserve(capacity);
+        array.resize(length);
+        PVStructurePtr * from = get();
+        for (size_t i=0; i<length; i++) array[i] = from[i];
+        value->swap(array);
+    } else {
+        value->reserve(capacity);
+    }
     setCapacityLength(capacity,length);
 }