Little bug fix on -opie option handling.

1 files changed, 50 insertions, 19 deletions

diff --git a/development/translation/opie-lrelease/main.cpp b/development/translation/opie-lrelease/main.cpp
index e5c1a0c..4c7352f 100644
--- a/development/translation/opie-lrelease/main.cpp
+++ b/development/translation/opie-lrelease/main.cpp
@@ -65,50 +65,72 @@ static void releaseQmFile( const QString& tsFileName, bool verbose )
 		 tsFileName.latin1() );
     }
 }
+
 static void metaQmFile( const QString &opiedir,
                         const QStringList& lang,
                         const QString& basename,
                         bool isLib, bool verb ) {
     QString target = basename + ".ts";
+
     if ( isLib ) target.prepend("lib");
 
-    for ( QStringList::ConstIterator it = lang.begin(); it != lang.end();
-          ++it ) {
+for ( QStringList::ConstIterator it = lang.begin(); it != lang.end(); ++it )
+{
         QString fileName = opiedir + "/i18n/" + (*it) + "/" + target;
         qWarning("Target is %s", fileName.latin1() );
 	releaseQmFile( fileName, verb );
     }
 }
+
 int main( int argc, char **argv )
 {
     bool verbose = FALSE;
-    bool metTranslations = FALSE;
+// bool metTranslations = FALSE;
     int numFiles = 0;
     QString opiedir;
     QStringList languageList = OPIE::self()->languageList( opiedir );
 
-    for ( int i = 1; i < argc; i++ ) {
-	if ( qstrcmp(argv[i], "-help") == 0 ) {
+for ( int i = 1; i < argc; i++ )
+{
+	if ( qstrcmp(argv[i], "-help") == 0 )
+	{
 	    printUsage();
 	    return 0;
-	} else if ( qstrcmp(argv[i], "-verbose") == 0 ) {
+	}
+	else if ( qstrcmp(argv[i], "-verbose") == 0 )
+	{
 	    verbose = TRUE;
 	    continue;
-	} else if ( qstrcmp(argv[i], "-version") == 0 ) {
+	}
+	else if ( qstrcmp(argv[i], "-version") == 0 )
+	{
 	    fprintf( stderr, "lrelease version %s\n", QT_VERSION_STR );
 	    return 0;
-	} else if ( qstrcmp(argv[i], "-opie") == 0 ) {
-            if ( i+1 < argc ) {
-                opiedir = argv[i+1];
+	}
+	else if ( qstrcmp(argv[i], "-opie") == 0 )
+	{
+		i++;
+		if ( i < argc )
+		{
+			opiedir = argv[i];
                 languageList = OPIE::self()->languageList(opiedir);
+			continue;
             }
+//		else
+//		{
+			fprintf( stderr, "lrelease error: -opie option need a parameter\n" );
+			printUsage();
+			return 1;
+//		}
         }
 
 	numFiles++;
 	QFile f( argv[i] );
-	if ( !f.open(IO_ReadOnly) ) {
+	if ( !f.open(IO_ReadOnly) )
+	{
 	    fprintf( stderr,
-		     "lrelease error: Cannot open file '%s': %s\n", argv[i],
+			 "lrelease error: Cannot open file '%s': %s\n",
+			 argv[i],
 		     strerror(errno) );
 	    return 1;
 	}
@@ -117,23 +139,31 @@ int main( int argc, char **argv )
 	QString fullText = t.read();
 	f.close();
 
-	if ( fullText.find(QString("<!DOCTYPE TS>")) >= 0 ) {
+	if ( fullText.find(QString("<!DOCTYPE TS>")) >= 0 )
+	{
 	    releaseQmFile( argv[i], verbose );
-	} else {
+	}
+	else
+	{
             QString target;
             bool isLib = FALSE;
 	    QMap<QString, QString> tagMap = proFileTagMap( fullText );
 	    QMap<QString, QString>::Iterator it;
 
-	    for ( it = tagMap.begin(); it != tagMap.end(); ++it ) {
+		for ( it = tagMap.begin(); it != tagMap.end(); ++it )
+		{
         	QStringList toks = QStringList::split( ' ', it.data() );
 		QStringList::Iterator t;
 
-        	for ( t = toks.begin(); t != toks.end(); ++t ) {
-		    if ( it.key() == "TARGET" ) {
+			for ( t = toks.begin(); t != toks.end(); ++t )
+			{
+				if ( it.key() == "TARGET" )
+				{
 			target = *t;
 			qWarning("%s %s", it.key().latin1(), (*t).latin1() );
-		    }else if ( it.key() == "TEMPLATE" ) {
+				}
+				else if ( it.key() == "TEMPLATE" )
+				{
                         if ( (*t).stripWhiteSpace().lower() == "lib" )
                             isLib = TRUE;
                     }
@@ -145,7 +175,8 @@ int main( int argc, char **argv )
 	}
     }
 
-    if ( numFiles == 0 ) {
+if ( numFiles == 0 )
+{
 	printUsage();
 	return 1;
     }