Merge remote branch 'md/master' into pvarr

* md/master:
  added
  minor changes
  comparison between implementations
  Proposed pvData.h interface
  added pvArray.html; made queue.h and bitSetUtil.* compatible with pvDataCPP.

pvDataApp/misc/queue.h

@@ -29,9 +29,9 @@ public:
     int getNumberFree();
     int getNumberUsed();
     queueElementPtr & getFree();
-    void setUsed(queueElementPtr &element);
+    void setUsed(queueElementPtr const &element);
     queueElementPtr & getUsed();
-    void releaseUsed(queueElementPtr &element);
+    void releaseUsed(queueElementPtr const &element);
 private:
     queueElementPtr nullElement;
     queueElementPtrArray elements;
@@ -92,7 +92,7 @@ std::tr1::shared_ptr<T> & Queue<T>::getFree()
 }
 
 template <typename T>
-void Queue<T>::setUsed(std::tr1::shared_ptr<T> &element)
+void Queue<T>::setUsed(std::tr1::shared_ptr<T> const &element)
 {
    if(element!=elements[nextSetUsed++]) {
         throw std::logic_error("not correct queueElement");
@@ -112,7 +112,7 @@ std::tr1::shared_ptr<T> & Queue<T>::getUsed()
 }
 
 template <typename T>
-void Queue<T>::releaseUsed(std::tr1::shared_ptr<T> &element)
+void Queue<T>::releaseUsed(std::tr1::shared_ptr<T> const &element)
 {
     if(element!=elements[nextReleaseUsed++]) {
         throw std::logic_error(

pvDataApp/pvMisc/bitSetUtil.cpp

@@ -13,25 +13,31 @@
 
 namespace epics { namespace pvData {
 
+using std::tr1::static_pointer_cast;
+using std::size_t;
+
 static bool checkBitSetPVField(
-    PVField *pvField,BitSet *bitSet,int32 initialOffset)
+    PVFieldPtr const &pvField,BitSetPtr const &bitSet,int32 initialOffset)
 {
-    bool atLeastOneBitSet = false;
-    bool allBitsSet = true;
     int32 offset = initialOffset;
     int32 nbits = pvField->getNumberFields();
     if(nbits==1) return bitSet->get(offset);
     int32 nextSetBit = bitSet->nextSetBit(offset);
     if(nextSetBit>=(offset+nbits)) return false;
+    if(nextSetBit<0) return false;
     if(bitSet->get(offset)) {
         if(nbits>1) {
             for(int32 i=offset+1; i<offset+nbits; i++) bitSet->clear(i);
         }
         return true;
     }
-    PVStructure *pvStructure = static_cast<PVStructure *>(pvField);
+
+    bool atLeastOneBitSet = false;
+    bool allBitsSet = true;
+    PVStructurePtr pvStructure = static_pointer_cast<PVStructure>(pvField);
+    offset = pvStructure->getFieldOffset() + 1;
     while(offset<initialOffset + nbits) {
-        PVField *pvSubField = pvStructure->getSubField(offset).get();
+        PVFieldPtr pvSubField = pvStructure->getSubField(offset);
         int32 nbitsNow = pvSubField->getNumberFields();
         if(nbitsNow==1) {
             if(bitSet->get(offset)) {
@@ -41,24 +47,16 @@ static bool checkBitSetPVField(
             }
             offset++;
         } else {
-            offset++;
-            PVStructure *pvSubStructure = static_cast<PVStructure*>(pvField);
-            PVFieldPtrArray pvSubStructureFields =
-                pvSubStructure->getPVFields();
-            int num = pvSubStructure->getStructure()->getNumberFields();
-            for(int32 i=0; i<num; i++) {
-                PVField *pvSubSubField = pvSubStructureFields[i].get();
-                bool result = checkBitSetPVField(pvSubSubField,bitSet,offset);
-                if(result) {
-                    atLeastOneBitSet = true;
-                    if(!bitSet->get(offset)) {
-                        allBitsSet = false;
-                    }
-                } else {
+            bool result = checkBitSetPVField(pvSubField,bitSet,offset);
+            if(result) {
+                atLeastOneBitSet = true;
+                if(!bitSet->get(offset)) {
                     allBitsSet = false;
                 }
-                offset += pvSubSubField->getNumberFields();
+            } else {
+                allBitsSet = false;
             }
+            offset += pvSubField->getNumberFields();
         }
     }
     if(allBitsSet) {
@@ -72,7 +70,7 @@ static bool checkBitSetPVField(
     return atLeastOneBitSet;
 }
 
-bool BitSetUtil::compress(BitSet *bitSet,PVStructure *pvStructure)
+bool BitSetUtil::compress(BitSetPtr const &bitSet,PVStructurePtr const &pvStructure)
 {
     return checkBitSetPVField(pvStructure,bitSet,0);   
 }

pvDataApp/pvMisc/bitSetUtil.h

@@ -17,7 +17,7 @@ namespace epics { namespace pvData {
 
 class BitSetUtil : private NoDefaultMethods {
 public:
-    static bool compress(BitSet *bitSet,PVStructure *pvStructure);
+    static bool compress(BitSetPtr const &bitSet,PVStructurePtr const &pvStructure);
 };
 
 }}