diff --git a/src/gui/seqmanager/SequenceManager.cpp b/src/gui/seqmanager/SequenceManager.cpp index 100916264..7bf9134d7 100644 --- a/src/gui/seqmanager/SequenceManager.cpp +++ b/src/gui/seqmanager/SequenceManager.cpp @@ -1899,7 +1899,7 @@ SequenceManager::setExportWavFile(const QString& fileName) } m_wavExporter = new WAVExporter(fileName); // If creation of the WAVExporter has failed, bail. - if (m_wavExporter->isComplete()) + if (!m_wavExporter->isOK()) return; // and install in the driver diff --git a/src/sound/WAVExporter.h b/src/sound/WAVExporter.h index 471da967d..85802cdf1 100644 --- a/src/sound/WAVExporter.h +++ b/src/sound/WAVExporter.h @@ -38,6 +38,11 @@ class WAVExporter { public: explicit WAVExporter(const QString& fileName); + /* + * Call this after the ctor to determine whether the file was + * successfully created. + */ + bool isOK() const { return static_cast(m_audioWriteStream); } /// called by the audio thread on start playback void start(); @@ -54,12 +59,6 @@ class WAVExporter /// Export is complete, or this object is not OK. /** * Called by the gui thread to request completion status. - * - * ??? update() could return a bool and then this could be changed to - * an isOK() that checks m_audioWriteStream. - * - * Call this after the ctor to determine whether the file was - * successfully created. true means failure. */ bool isComplete() const;