Changeset 260 in flair-src for trunk/tools/FlairGCS/src/Manager.cpp

Timestamp:

Jul 26, 2018, 2:44:42 PM (7 years ago)

Author:

Sanahuja Guillaume

Message:

gcs: do not use socket if connection lost

File:

• trunk/tools/FlairGCS/src/Manager.cpp (modified) (3 diffs)

trunk/tools/FlairGCS/src/Manager.cpp

@@ -154 +154 @@
   connect(newLayout, SIGNAL(destroyed(QObject *)), this, SLOT(layoutDestroyed(QObject *)));
   connect(udtSocket, SIGNAL(destroyed(QObject *)), this, SLOT(udtSocketDestroyed(QObject *)));
+  connect(udtSocket, SIGNAL(destroyed(QObject *)), newLayout, SLOT(udtSocketDestroyed(QObject *)));
   
   // widget
@@ -187 +188 @@
 void Manager::udtSocketDestroyed(QObject *obj) {
   //mark socket as not valid, it can be a connection lost
-  //in this case we keep the layout for reading, and mark it as problematic
+  //in this case we keep the layout for reading, and mark it as problematic (socket=NULL)
   for(int i=0;i<connections.count();i++) {
     if(connections.at(i).socket==(UdtSocket *)obj) {
@@ -391 +392 @@
 void Manager::send(void) {
   for (int i = 0; i < connections.count(); i++) {
-    QDomDocument doc("remote_ui_xml");
-    connections.at(i).layout->GetUpdateXml((QDomElement *)&doc);
-    // fprintf(stderr,"merge\n%s\n",doc.toString().toLocal8Bit().constData());
-
-    connections.at(i).layout->XmlToSend(doc);
+    if(connections.at(i).socket!=NULL) {
+      QDomDocument doc("remote_ui_xml");
+      connections.at(i).layout->GetUpdateXml((QDomElement *)&doc);
+      // fprintf(stderr,"merge\n%s\n",doc.toString().toLocal8Bit().constData());
+
+      connections.at(i).layout->XmlToSend(doc);
+    }
   }
 }