make thetime

not a pointer
and memset it
this makes valgrind shut up

1 files changed, 17 insertions, 16 deletions

diff --git a/library/backend/timeconversion.cpp b/library/backend/timeconversion.cpp
index ef7762d..3c25922 100644
--- a/library/backend/timeconversion.cpp
+++ b/library/backend/timeconversion.cpp
@@ -128,17 +128,17 @@ QDateTime TimeConversion::fromISO8601( const QCString &s )
 {

 

 #if defined(_OS_WIN32) || defined (Q_OS_WIN32) || defined (Q_OS_WIN64)

     _tzset();

 #else

     tzset();

 #endif

 

-    struct tm *thetime = new tm;

+    struct tm thetime;

 

     QCString str = s.copy();

     str.replace(QRegExp("-"), "" );

     str.replace(QRegExp(":"), "" );

     str.stripWhiteSpace();

     str = str.lower();

 

     int i = str.find( "t" );

@@ -147,33 +147,34 @@ QDateTime TimeConversion::fromISO8601( const QCString &s )
     if ( i != -1 ) {

 	date = str.left( i );

 	timestr = str.mid( i+1 );

     } else {

 	date = str;

     }

 

 //     qDebug("--- parsing ISO time---");

-    thetime->tm_year = 100;

-    thetime->tm_mon = 0;

-    thetime->tm_mday = 0;

-    thetime->tm_hour = 0;

-    thetime->tm_min = 0;

-    thetime->tm_sec = 0;

+    memset( &thetime, 0, sizeof(tm) );

+    thetime.tm_year = 100;

+    thetime.tm_mon = 0;

+    thetime.tm_mday = 0;

+    thetime.tm_hour = 0;

+    thetime.tm_min = 0;

+    thetime.tm_sec = 0;

 

 //     qDebug("date = %s", date.data() );

 

     switch( date.length() ) {

 	case 8:

-	    thetime->tm_mday = date.right( 2 ).toInt();

+	    thetime.tm_mday = date.right( 2 ).toInt();

 	case 6:

-	    thetime->tm_mon = date.mid( 4, 2 ).toInt() - 1;

+	    thetime.tm_mon = date.mid( 4, 2 ).toInt() - 1;

 	case 4:

-	    thetime->tm_year = date.left( 4 ).toInt();

-	    thetime->tm_year -= 1900;

+	    thetime.tm_year = date.left( 4 ).toInt();

+	    thetime.tm_year -= 1900;

 	    break;

 	default:

 	    break;

     }

 

     int tzoff = 0;

     bool inLocalTime = FALSE;

     if ( timestr.find( 'z' ) == (int)timestr.length() - 1 )

@@ -201,37 +202,37 @@ QDateTime TimeConversion::fromISO8601( const QCString &s )
 	    tzoff = 60*tzoffhour + tzoffmin;

 	} else

 	    inLocalTime = TRUE;

     }

 

     // get the time:

     switch( timestr.length() ) {

 	case 6:

-	    thetime->tm_sec = timestr.mid( 4 ).toInt();

+	    thetime.tm_sec = timestr.mid( 4 ).toInt();

 	case 4:

-	    thetime->tm_min = timestr.mid( 2, 2 ).toInt();

+	    thetime.tm_min = timestr.mid( 2, 2 ).toInt();

 	case 2:

-	    thetime->tm_hour = timestr.left( 2 ).toInt();

+	    thetime.tm_hour = timestr.left( 2 ).toInt();

 	default:

 	    break;

     }

 

     int tzloc = 0;

     time_t tmp = time( 0 );

     if ( !inLocalTime ) {

 	// have to get the offset between gmt and local time

 	struct tm *lt = localtime( &tmp );

 	tzloc = mktime( lt );

 	struct tm *ut = gmtime( &tmp );

 	tzloc -= mktime( ut );

     }

 //     qDebug("time: %d %d %d, tzloc=%d, tzoff=%d", thetime->tm_hour, thetime->tm_min, thetime->tm_sec,

 //  	   tzloc, tzoff );

 

-    tmp = mktime( thetime );

+    tmp = mktime( &thetime );

     tmp += 60*(-tzloc + tzoff);

 

-    delete thetime;

+

 

     return fromUTC( tmp );

 }