[Kstars-devel] kdeedu/kstars/kstars [POSSIBLY UNSAFE]

[Jasem Mutlaq]

CVS commit by mutlaqja: 

Fixing an issue with Celestron driver reported by Mr. Quinn Jensen. The delay between two consequetive slews now is 500 ms. The reason that KStars was sending two consequetive slews is because I forgot to add 'break' in a switch statement. The issue is fixed now and the driver can handle fast slews properly. Thanks to Quinn Jensen for the patch!

CCMAIL: kstars-devel at kde.org, quinn at zdomain.com


  M +3 -2      indidevice.cpp   1.14 [POSSIBLY UNSAFE: printf]
  M +2 -0      indidriver.cpp   1.14
  M +2 -2      indi/celestrongps.cpp   1.5


--- kdeedu/kstars/kstars/indidevice.cpp  #1.13:1.14
@@ -351,4 +351,5 @@ void INDI_P::convertSwitch(int id)
                 newSwitch(switchIndex);
            prop->newText();
+           break;
 
          case PP_WO:
@@ -364,7 +365,7 @@ void INDI_P::convertSwitch(int id)
         return;
    }
-   else if ((lp->name == "Abort Slew/Track"))
+   else if ((lp->name == "Abort"))
    {
-         kdDebug() << "Stopping timer " << endl;
+         fprintf(stderr, "Stopping timer.\n");
          pg->dp->devTimer->stop();
          if (lp->state == PS_OFF)

--- kdeedu/kstars/kstars/indidriver.cpp  #1.13:1.14
@@ -126,4 +126,6 @@ for (uint i=0; i < ksw->data()->INDIHost
         ksw->data()->INDIHostsList.at(i)->isConnected = false;
         affectedItem->setPixmap(0, disconnected);
+        connectHostB->setEnabled(true);
+        disconnectHostB->setEnabled(false);
         break;
      }

--- kdeedu/kstars/kstars/indi/celestrongps.cpp  #1.4:1.5
@@ -191,6 +191,6 @@ int CelestronGPS::handleCoordSet()
              StopNSEW();
 
-             // sleep for 100 mseconds
-             usleep(100000);
+             // sleep for 500 mseconds
+             usleep(500000);
           }