fixed neg lookuptime bug

1 files changed, 115 insertions, 113 deletions

diff --git a/noncore/settings/netsystemtime/ntp.cpp b/noncore/settings/netsystemtime/ntp.cpp
index f60e38d..40805a0 100644
--- a/noncore/settings/netsystemtime/ntp.cpp
+++ b/noncore/settings/netsystemtime/ntp.cpp
@@ -29,20 +29,20 @@
 
 Ntp::Ntp( QWidget* parent,  const char* name, WFlags fl )
-    : SetDateTime( parent, name, fl )
+  : SetDateTime( parent, name, fl )
 {
-	Config ntpSrvs(QPEApplication::qpeDir()+"etc/ntpservers",Config::File);
+  Config ntpSrvs(QPEApplication::qpeDir()+"etc/ntpservers",Config::File);
   ntpSrvs.setGroup("servers");
   int srvCount = ntpSrvs.readNumEntry("count", 0 );
   for (int i = 0; i < srvCount; i++)
-  {
-	  ntpSrvs.setGroup(QString::number(i));
-   	ComboNtpSrv->insertItem( ntpSrvs.readEntry("name") );
-  }
+    {
+      ntpSrvs.setGroup(QString::number(i));
+      ComboNtpSrv->insertItem( ntpSrvs.readEntry("name") );
+    }
   if ( srvCount==0 ) ComboNtpSrv->insertItem(tr("time.fu-berlin.de"));
 
-	Config cfg("ntp",Config::User);
+  Config cfg("ntp",Config::User);
   cfg.setGroup("settings");
-	SpinBoxMinLookupDelay->setValue( cfg.readNumEntry("minLookupDiff",720) );
-	SpinBoxNtpDelay->setValue( cfg.readNumEntry("ntpRefreshFreq",1440) );
+  SpinBoxMinLookupDelay->setValue( cfg.readNumEntry("minLookupDiff",720) );
+  SpinBoxNtpDelay->setValue( cfg.readNumEntry("ntpRefreshFreq",1440) );
   ComboNtpSrv->setCurrentItem( cfg.readNumEntry("ntpServer", 0) );
 
@@ -57,17 +57,17 @@ Ntp::Ntp( QWidget* parent,  const char* name, WFlags fl )
   ntpTimer = new QTimer(this);
 
- 	ntpProcess = new OProcess( );
+  ntpProcess = new OProcess( );
   connect( SpinBoxNtpDelay, SIGNAL( valueChanged(int) ),
-  					SLOT(slotNtpDelayChanged(int)) );
+	   SLOT(slotNtpDelayChanged(int)) );
 
   ntpSock = new QSocket( this );
   connect( ntpSock, SIGNAL( error(int) ),
-  					SLOT(slotCheckNtp(int)) );
+	   SLOT(slotCheckNtp(int)) );
   slotProbeNtpServer();
 
   connect ( ntpProcess, SIGNAL(receivedStdout(OProcess*,char*,int)),
-  					this, SLOT(getNtpOutput(OProcess*,char*,int)));
+	    this, SLOT(getNtpOutput(OProcess*,char*,int)));
   connect ( ntpProcess, SIGNAL(processExited(OProcess*)),
-  					this, SLOT(ntpFinished(OProcess*)));
+	    this, SLOT(ntpFinished(OProcess*)));
   connect(runNtp, SIGNAL(clicked()), this, SLOT(slotRunNtp()));
   connect(PushButtonPredict, SIGNAL(clicked()), this, SLOT(preditctTime()));
@@ -98,13 +98,15 @@ Ntp::~Ntp()
 bool Ntp::ntpDelayElapsed()
 {
-	Config cfg("ntp",Config::User);
+  Config cfg("ntp",Config::User);
   cfg.setGroup("lookups");
   _lookupDiff = TimeConversion::toUTC(QDateTime::currentDateTime()) - cfg.readNumEntry("time",0);
-  return (_lookupDiff - (SpinBoxNtpDelay->value()*60)) > -60;
+  if (_lookupDiff < 0) return true; 
+  int i =_lookupDiff - (SpinBoxNtpDelay->value()*60);
+  return i > -60;
 }
 
 QString Ntp::getNtpServer()
 {
-	return ComboNtpSrv->currentText();
+  return ComboNtpSrv->currentText();
 }
 
@@ -112,26 +114,26 @@ void Ntp::slotRunNtp()
 {
   if ( !ntpDelayElapsed() )
-  {
-  	switch (
-   	 QMessageBox::warning(this, tr("Run NTP?"),
-     tr("You asked for a delay of ")+SpinBoxNtpDelay->text()+tr(" minutes, but only ")+
-     QString::number(_lookupDiff/60)+tr(" minutes elapsed since last lookup.")+
-     "<br>"+tr("Rerun NTP?"),
-     QMessageBox::Ok,QMessageBox::Cancel)
-   	) {
-	    	case QMessageBox::Ok:     break;
-  	   	case QMessageBox::Cancel: return;
-    	  default: return;
+    {
+      switch (
+	      QMessageBox::warning(this, tr("Run NTP?"),
+				   tr("You asked for a delay of ")+SpinBoxNtpDelay->text()+tr(" minutes, but only ")+
+				   QString::number(_lookupDiff/60)+tr(" minutes elapsed since last lookup.")+
+				   "<br>"+tr("Rerun NTP?"),
+				   QMessageBox::Ok,QMessageBox::Cancel)
+	      ) {
+      case QMessageBox::Ok:     break;
+      case QMessageBox::Cancel: return;
+      default: return;
       }
-  }
-	TextLabelStartTime->setText(QDateTime::currentDateTime().toString());
+    }
+  TextLabelStartTime->setText(QDateTime::currentDateTime().toString());
   ntpOutPut( tr("Running:")+"\nntpdate "+getNtpServer() );
  	
-	ntpProcess->clearArguments();
-	*ntpProcess << "ntpdate" << getNtpServer();
-	bool ret = ntpProcess->start(OProcess::NotifyOnExit,OProcess::AllOutput);
+  ntpProcess->clearArguments();
+  *ntpProcess << "ntpdate" << getNtpServer();
+  bool ret = ntpProcess->start(OProcess::NotifyOnExit,OProcess::AllOutput);
   if ( !ret ) {
-     qDebug("Error while executing ntpdate");
-     ntpOutPut( tr("Error while executing ntpdate"));
+    qDebug("Error while executing ntpdate");
+    ntpOutPut( tr("Error while executing ntpdate"));
   }
 }
@@ -143,8 +145,8 @@ void  Ntp::getNtpOutput(OProcess *proc, char *buffer, int buflen)
   lineStr=lineStr.left(buflen);
   if (lineStr!=lineStrOld)
-  {
-	        ntpOutPut(lineStr);
-         	_ntpOutput += lineStr;
-  }
+    {
+      ntpOutPut(lineStr);
+      _ntpOutput += lineStr;
+    }
   lineStrOld = lineStr;
 }
@@ -153,9 +155,9 @@ void  Ntp::ntpFinished(OProcess *p)
 {
   //	qDebug("p->exitStatus() %i",p->exitStatus());
-	if (p->exitStatus()!=0 || !p->normalExit())
- 	{
-    slotProbeNtpServer();
-   	return;
-  }
+  if (p->exitStatus()!=0 || !p->normalExit())
+    {
+      slotProbeNtpServer();
+      return;
+    }
 
   Global::writeHWClock();
@@ -165,8 +167,8 @@ void  Ntp::ntpFinished(OProcess *p)
   DateBookDB db;
   
-//  QCopEnvelope timeApplet( "QPE/TaskBar", "reloadApplets()" );
-//  timeApplet << "";
+  //  QCopEnvelope timeApplet( "QPE/TaskBar", "reloadApplets()" );
+  //  timeApplet << "";
   
-	Config cfg("ntp",Config::User);
+  Config cfg("ntp",Config::User);
   cfg.setGroup("lookups");
   int lastLookup = cfg.readNumEntry("time",0);
@@ -178,19 +180,19 @@ void  Ntp::ntpFinished(OProcess *p)
   float timeShift = getTimeShift();
   if (timeShift == 0.0) return;
- 	int secsSinceLast = time - lastLookup;
+  int secsSinceLast = time - lastLookup;
   TextLabelNewTime->setText(QDateTime::currentDateTime().toString());
   TextLabelTimeShift->setText(QString::number(timeShift)+tr(" seconds"));
   if ( lastNtp && lastLookup > 0 && secsSinceLast > 60* SpinBoxMinLookupDelay->value())
-  {
-	  cfg.setGroup("lookup_"+QString::number(lookupCount));
-	  lookupCount++;
-    _shiftPerSec =  timeShift / secsSinceLast;
-   	qDebug("secs since last lookup %i", secsSinceLast);qDebug("timeshift since last lookup %f", timeShift);qDebug("timeshift since per sec %f", _shiftPerSec);
-		cfg.writeEntry("secsSinceLast",secsSinceLast);
-		cfg.writeEntry("timeShift",QString::number(timeShift));
-	  cfg.setGroup("lookups");
-   	cfg.writeEntry("count",lookupCount);
-  	cfg.writeEntry("lastNtp",true);
-  }
+    {
+      cfg.setGroup("lookup_"+QString::number(lookupCount));
+      lookupCount++;
+      _shiftPerSec =  timeShift / secsSinceLast;
+      qDebug("secs since last lookup %i", secsSinceLast);qDebug("timeshift since last lookup %f", timeShift);qDebug("timeshift since per sec %f", _shiftPerSec);
+      cfg.writeEntry("secsSinceLast",secsSinceLast);
+      cfg.writeEntry("timeShift",QString::number(timeShift));
+      cfg.setGroup("lookups");
+      cfg.writeEntry("count",lookupCount);
+      cfg.writeEntry("lastNtp",true);
+    }
 }
 
@@ -198,5 +200,5 @@ void  Ntp::ntpFinished(OProcess *p)
 float Ntp::getTimeShift()
 {
-	QString _offset = "offset";
+  QString _offset = "offset";
   QString _sec = "sec";
   QRegExp _reOffset = QRegExp(_offset);
@@ -213,5 +215,5 @@ float Ntp::getTimeShift()
 void Ntp::readLookups()
 {
-	Config cfg("ntp",Config::User);
+  Config cfg("ntp",Config::User);
   cfg.setGroup("lookups");
   int lookupCount = cfg.readNumEntry("count",0);
@@ -230,18 +232,18 @@ void Ntp::readLookups()
   TableLookups->setColumnWidth( 2, cw );
   TableLookups->sortColumn(0, false, true );
- // TableLookups->setSorting( true );
+  // TableLookups->setSorting( true );
   _shiftPerSec = 0;
   for (int i=0; i < lookupCount; i++)
-  {
-	  cfg.setGroup("lookup_"+QString::number(i));
-   	last = cfg.readEntry("secsSinceLast",0).toFloat();
-    shift = QString(cfg.readEntry("timeShift",0)).toFloat();
-    shiftPerSec =  shift / last;
-   	qDebug("%i shift %f",i,shiftPerSec);
-    _shiftPerSec += shiftPerSec;
-		TableLookups->setText( i,0,QString::number(shiftPerSec*60*60));
-		TableLookups->setText( i,2,QString::number(shift));
-	  TableLookups->setText( i,1,QString::number(last/(60*60)));
-  }
+    {
+      cfg.setGroup("lookup_"+QString::number(i));
+      last = cfg.readEntry("secsSinceLast",0).toFloat();
+      shift = QString(cfg.readEntry("timeShift",0)).toFloat();
+      shiftPerSec =  shift / last;
+      qDebug("%i shift %f",i,shiftPerSec);
+      _shiftPerSec += shiftPerSec;
+      TableLookups->setText( i,0,QString::number(shiftPerSec*60*60));
+      TableLookups->setText( i,2,QString::number(shift));
+      TableLookups->setText( i,1,QString::number(last/(60*60)));
+    }
   _shiftPerSec /= lookupCount;
   TextLabelShift->setText(QString::number(_shiftPerSec*60*60)+tr(" s/h"));
@@ -250,8 +252,8 @@ void Ntp::readLookups()
 void Ntp::preditctTime()
 {
-	Config cfg("ntp",Config::User);
+  Config cfg("ntp",Config::User);
   cfg.setGroup("lookups");
- 	int lastTime = cfg.readNumEntry("time",0);
- 	cfg.writeEntry("lastNtp",true);
+  int lastTime = cfg.readNumEntry("time",0);
+  cfg.writeEntry("lastNtp",true);
   setenv( "TZ", tz->currentZone(), 1 );
   int now = TimeConversion::toUTC( QDateTime::currentDateTime() );
@@ -259,5 +261,5 @@ void Ntp::preditctTime()
   TextLabelEstimatedShift->setText(QString::number(corr)+tr(" seconds"));
   predictedTime = QDateTime::currentDateTime().addSecs(corr);
- 	TextLabelPredTime->setText(predictedTime.toString());
+  TextLabelPredTime->setText(predictedTime.toString());
   TextLabelMainPredTime->setText(tr("Predicted time:")+"<br><b>"+predictedTime.toString()+"</b>");
 }
@@ -266,35 +268,35 @@ void Ntp::setPredictTime()
 {
   preditctTime();
- 	setTime( predictedTime );
+  setTime( predictedTime );
 }
 
 void Ntp::slotCheckNtp(int i)
 {
-	if (i == 0)
- 	{
-    TextLabelMainPredTime->hide();
-    ButtonSetTime->setText( tr("Get time from network") );
-	  connect( ButtonSetTime, SIGNAL(clicked()), SLOT(slotRunNtp()) );
-	 	if ( ntpDelayElapsed() )
+  if (i == 0)
+    {
+      TextLabelMainPredTime->hide();
+      ButtonSetTime->setText( tr("Get time from network") );
+      connect( ButtonSetTime, SIGNAL(clicked()), SLOT(slotRunNtp()) );
+      if ( ntpDelayElapsed() )
    	{
-      slotRunNtp();
-	  	disconnect(ntpTimer, SIGNAL( timeout() ), this, SLOT(slotProbeNtpServer()) );
-    	connect(ntpTimer, SIGNAL( timeout() ), SLOT(slotRunNtp()) );
-	  }else{
-    	disconnect(ntpTimer, SIGNAL( timeout() ), this, SLOT(slotRunNtp()) );
-		  connect(ntpTimer, SIGNAL( timeout() ), SLOT(slotProbeNtpServer()) );
+	  slotRunNtp();
+	  disconnect(ntpTimer, SIGNAL( timeout() ), this, SLOT(slotProbeNtpServer()) );
+	  connect(ntpTimer, SIGNAL( timeout() ), SLOT(slotRunNtp()) );
+	}else{
+	  disconnect(ntpTimer, SIGNAL( timeout() ), this, SLOT(slotRunNtp()) );
+	  connect(ntpTimer, SIGNAL( timeout() ), SLOT(slotProbeNtpServer()) );
+	}
+    }else{
+      preditctTime();
+      ButtonSetTime->setText( tr("Set predicted time: ")+predictedTime.toString() );
+      if (i>0)ntpOutPut(tr("Could not connect to server ")+getNtpServer());
+      connect( ButtonSetTime, SIGNAL(clicked()), SLOT(setPredictTime()) );
+      connect( ntpTimer, SIGNAL( timeout() ), SLOT(slotProbeNtpServer()) );
     }
-  }else{
-	  preditctTime();
-	  ButtonSetTime->setText( tr("Set predicted time: ")+predictedTime.toString() );
-  	if (i>0)ntpOutPut(tr("Could not connect to server ")+getNtpServer());
-  	connect( ButtonSetTime, SIGNAL(clicked()), SLOT(setPredictTime()) );
-	  connect( ntpTimer, SIGNAL( timeout() ), SLOT(slotProbeNtpServer()) );
-  }
 }
 
 void Ntp::slotProbeNtpServer()
 {
-	ntpSock->connectToHost( getNtpServer() ,123);
+  ntpSock->connectToHost( getNtpServer() ,123);
 }
 
@@ -306,5 +308,5 @@ void Ntp::slotNtpDelayChanged(int delay)
 void Ntp::ntpOutPut(QString out)
 {
-	MultiLineEditntpOutPut->append(out);
+  MultiLineEditntpOutPut->append(out);
   MultiLineEditntpOutPut->setCursorPosition(MultiLineEditntpOutPut->numLines() + 1,0,FALSE);
 }
@@ -313,7 +315,7 @@ void Ntp::ntpOutPut(QString out)
 void Ntp::makeChannel()
 {   	
-	channel = new QCopChannel( "QPE/Application/netsystemtime", this );
-	connect( channel, SIGNAL(received(const QCString&, const QByteArray&)),
-		this, SLOT(receive(const QCString&, const QByteArray&)) );
+  channel = new QCopChannel( "QPE/Application/netsystemtime", this );
+  connect( channel, SIGNAL(received(const QCString&, const QByteArray&)),
+	   this, SLOT(receive(const QCString&, const QByteArray&)) );
 }
 
@@ -322,15 +324,15 @@ void Ntp::makeChannel()
 void Ntp::receive(const QCString &msg, const QByteArray &arg)
 {
-	qDebug("QCop(Ntp) "+msg+" "+QCString(arg));
-	if ( msg == "ntpLookup(QString)" )
- 	{
-  	  slotRunNtp();
-	}
-	if ( msg == "setPredictedTime(QString)" )
- 	{
-  	  setPredictTime();
-	}else{
-    	qDebug("Ntp::receive: Huh what do ya want");
- 	}
+  qDebug("QCop(Ntp) "+msg+" "+QCString(arg));
+  if ( msg == "ntpLookup(QString)" )
+    {
+      slotRunNtp();
+    }
+  if ( msg == "setPredictedTime(QString)" )
+    {
+      setPredictTime();
+    }else{
+      qDebug("Ntp::receive: Huh what do ya want");
+    }
 }