Changed Module::readAll() to read all push-type variables first, to make sure...

Changed Module::readAll() to read all push-type variables first, to make sure that poll-type variables are up-to-date upon return

include/Module.h

@@ -50,8 +50,9 @@ namespace ChimeraTK {
        *  all variables. The returned TransferElement will be the push-type variable which has been updated. */
       boost::shared_ptr<mtca4u::TransferElement> readAny();
 
-      /** Just call read() on all readable variables in the group. If there are push-type variables in the group, this
-       *  call will block until all of the variables have received an update. */
+      /** Read all readable variables in the group. If there are push-type variables in the group, this call will block
+       *  until all of the variables have received an update. All push-type variables are read first, the poll-type
+       *  variables are therefore updated with the latest values upon return. */
       void readAll();
 
       /** Just call readNonBlocking() on all readable variables in the group. */

src/Module.cc

@@ -79,10 +79,18 @@ namespace ChimeraTK {
   
   void Module::readAll() {
     auto accessorList = getAccessorListRecursive();
+    // first blockingly read all push-type variables
     for(auto accessor : accessorList) {
       if(!accessor.getAppAccessorNoType().isReadable()) continue;
+      if(accessor.getMode() != UpdateMode::push) continue;
       accessor.getAppAccessorNoType().read();
     }
+    // next non-blockingly read the latest values of all poll-type variables
+    for(auto accessor : accessorList) {
+      if(!accessor.getAppAccessorNoType().isReadable()) continue;
+      if(accessor.getMode() == UpdateMode::push) continue;
+      accessor.getAppAccessorNoType().readLatest();
+    }
   }
 
 /*********************************************************************************************************************/