Changeset 139 in flair-src for trunk/lib/FlairSensorActuator

Timestamp:

02/14/17 17:02:06 (7 years ago)

Author:

Sanahuja Guillaume

Message:

warn message if vrpnobject is added on running client

Location:

trunk/lib/FlairSensorActuator/src

Files:

• VrpnClient_impl.cpp (modified) (1 diff)
• VrpnObject_impl.cpp (modified) (3 diffs)

trunk/lib/FlairSensorActuator/src/VrpnClient_impl.cpp

@@ -215 +215 @@
       }
     } else {//!UseXbee()
-                        if(connection->connected()==vrpn_true && !isConnected) {
-                                isConnected=true;
-                                Printf("VRPN connected\n");
-                        }
-                        if(connection->connected()==vrpn_false && isConnected) {
-                                isConnected=false;
-                                Printf("VRPN disconnected\n");
-                        }
-                        
-                        //timeout in mainloop is not well handled if not connected...
-                        if(isConnected) {
-                                //this is when trackables callbacks are called:
-                                if(connection->mainloop(&timeout)!=0) {
-                                        self->Warn("connection dropped\n");
-                                }
-                                //printf("%lld\n",GetTime()/(1000*1000));
-                                mutex->GetMutex();
-                                for (unsigned int i = 0; i < trackables.size(); i++)
-                                        trackables.at(i)->mainloop();
-                                mutex->ReleaseMutex();
-                        } else {
-                                connection->mainloop();
-                                self->SleepMS(10);
-                        }
+        if(connection->connected()==vrpn_true && !isConnected) {
+            isConnected=true;
+            Printf("VRPN connected\n");
+        }
+        if(connection->connected()==vrpn_false && isConnected) {
+            isConnected=false;
+            Printf("VRPN disconnected\n");
+        }
+        //timeout in mainloop is not well handled if not connected...
+        if(isConnected) {
+            //this is when trackables callbacks are called:
+            if(connection->mainloop(&timeout)!=0) {
+                self->Warn("connection dropped\n");
+            }
+            //printf("%lld\n",GetTime()/(1000*1000));
+            mutex->GetMutex();
+            for (unsigned int i = 0; i < trackables.size(); i++)
+                trackables.at(i)->mainloop();
+            mutex->ReleaseMutex();
+        } else {
+            connection->mainloop();
+            self->SleepMS(10);
+        }
     }
   }

trunk/lib/FlairSensorActuator/src/VrpnObject_impl.cpp

@@ -56 +56 @@
   }
 
-  if (parent->UseXbee()) {
-    parent->pimpl_->AddTrackable(this, id);
-    tracker = NULL;
-  } else {
-    parent->pimpl_->AddTrackable(self);
-    tracker = new vrpn_Tracker_Remote(name.c_str(), parent->pimpl_->connection);
-    tracker->register_change_handler(this, handle_pos);
-    tracker->shutup = true;
-  }
-
   // state
   cvmatrix_descriptor *desc = new cvmatrix_descriptor(7, 1);
@@ -91 +81 @@
   z_plot = new DataPlot1D(plot_tab->LastRowLastCol(), "z", -2, 0);
   z_plot->AddCurve(output->Element(6));
+  
+  if(parent->IsRunning()) {
+    self->Warn("adding VrpnObject on running VrpnClient can crash\n");
+  }
+  
+  if (parent->UseXbee()) {
+    tracker = NULL;
+    parent->pimpl_->AddTrackable(this, id);
+  } else {
+    tracker = new vrpn_Tracker_Remote(name.c_str(), parent->pimpl_->connection);
+    tracker->register_change_handler(this, handle_pos);
+    tracker->shutup = true;
+    parent->pimpl_->AddTrackable(self);
+  }
 }
 
@@ -106 +110 @@
 }
 
-void VrpnObject_impl::mainloop(void) { tracker->mainloop(); }
+void VrpnObject_impl::mainloop(void) { 
+     tracker->mainloop();
+}
 
 bool VrpnObject_impl::IsTracked(unsigned int timeout_ms) {