[Traverso-commit] traverso/src/commands AudioClipExternalProcessi...

[Remon Sijrier]

CVSROOT:        /sources/traverso
Module name:    traverso
Changes by:     Remon Sijrier <r_sijrier>       07/07/02 20:26:47

Modified files:
        src/commands   : AudioClipExternalProcessing.cpp 

Log message:
        detect crash finish, don't count on the statusbar to see if the 
external render finished 

CVSWeb URLs:
http://cvs.savannah.gnu.org/viewcvs/traverso/src/commands/AudioClipExternalProcessing.cpp?cvsroot=traverso&r1=1.17&r2=1.18

Patches:
Index: AudioClipExternalProcessing.cpp
===================================================================
RCS file: 
/sources/traverso/traverso/src/commands/AudioClipExternalProcessing.cpp,v
retrieving revision 1.17
retrieving revision 1.18
diff -u -b -r1.17 -r1.18
--- AudioClipExternalProcessing.cpp     2 Jul 2007 19:16:48 -0000       1.17
+++ AudioClipExternalProcessing.cpp     2 Jul 2007 20:26:46 -0000       1.18
@@ -279,8 +279,11 @@
                foreach(QString token, tokens) {
                        if (token.contains("%")) {
                                token = token.remove("%)");
-                               int number = (int)token.toDouble();
+                               bool ok;
+                               int number = (int)token.toDouble(&ok);
+                               if (ok) {
                                progressBar->setValue(number);
+                               }
                                return;
                        }
                }
@@ -304,17 +307,16 @@
                return;
        }
        
+       if (exitstatus == QProcess::CrashExit) {
+               statusText->setHtml(tr("Program <b>%1</b> 
crashed!").arg(m_program));
+               return;
+       }
+       
        QString dir = pm().get_project()->get_audiosources_dir();
        
        // In case we used the merger, remove the file...
        QFile::remove(dir + "/merged.wav");
        
-       if (progressBar->value() != 100) {
-               // not sure if this is always valid, but it at least should be 
100 all 
-               // the time, that is, after succesfull operation....
-               // so if not 100 -> unsucesfull, and we bail out
-               return;
-       }
        
        QString result = m_processor->readAllStandardOutput();
        // print anything on command line we didn't catch