Unify the canvases by moving groups around rather than using set_scrolling_region...
[ardour.git] / gtk2_ardour / editor_audio_import.cc
index f2873f3f0bd286ec486efbbfa3ec1c039251446b..3d75de782c5c2017884d8af5c4c8cf0002287c25 100644 (file)
 
 #include <sys/types.h>
 #include <sys/stat.h>
+#include <sys/time.h>
 #include <errno.h>
 #include <unistd.h>
+#include <algorithm>
 
 #include <sndfile.h>
 
@@ -41,6 +43,7 @@
 #include <ardour/audiofilesource.h>
 #include <ardour/region_factory.h>
 #include <ardour/source_factory.h>
+#include <ardour/session.h>
 #include <pbd/memento_command.h>
 
 #include "ardour_ui.h"
@@ -64,70 +67,188 @@ using Glib::ustring;
 /* Functions supporting the incorporation of external (non-captured) audio material into ardour */
 
 void
-Editor::add_external_audio_action (ImportMode mode)
+Editor::add_external_audio_action (ImportMode mode_hint)
 {
+       if (session == 0) {
+               MessageDialog msg (_("You can't import or embed an audiofile until you have a session loaded."));
+               msg.run ();
+               return;
+       }
+       
+       if (sfbrowser == 0) {
+               sfbrowser = new SoundFileOmega (*this, _("Add existing audio"), session, 0, true, mode_hint);
+       } else {
+               sfbrowser->set_mode (mode_hint);
+       }
+
+       external_audio_dialog ();
 }
 
 void
 Editor::external_audio_dialog ()
 {
        vector<Glib::ustring> paths;
+       uint32_t track_cnt;
 
        if (session == 0) {
-               MessageDialog msg (0, _("You can't import or embed an audiofile until you have a session loaded."));
+               MessageDialog msg (_("You can't import or embed an audiofile until you have a session loaded."));
                msg.run ();
                return;
        }
        
+       track_cnt = 0;
+
+       for (TrackSelection::iterator x = selection->tracks.begin(); x != selection->tracks.end(); ++x) {
+               AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(*x);
+               
+               if (!atv) {
+                       continue;
+               } else if (atv->is_audio_track()) {
+                       track_cnt++;
+               }
+       }
+
        if (sfbrowser == 0) {
-               sfbrowser = new SoundFileBrowser (*this, _("Add existing audio"), session, selection->tracks.size());
+               sfbrowser = new SoundFileOmega (*this, _("Add existing audio"), session, track_cnt, true);
        } else {
-               sfbrowser->reset (selection->tracks.size());
+               sfbrowser->reset (track_cnt);
        }
 
        sfbrowser->show_all ();
 
-       int response = sfbrowser->run ();
 
-       sfbrowser->hide ();
+       bool keepRunning;
 
-       switch (response) {
-       case RESPONSE_OK:
-               break;
-       default:
-               // cancel from the browser - we are done
-               return;
-       }
+       do {
+               keepRunning = false;
 
-       /* lets do it */
-       
-       paths = sfbrowser->get_paths ();
+               int response = sfbrowser->run ();
 
-       ImportPosition pos = sfbrowser->get_position ();
-       ImportMode mode = sfbrowser->get_mode ();
-       ImportDisposition chns = sfbrowser->get_channel_disposition ();
-       nframes64_t where;
+               switch (response) {
+                       case RESPONSE_APPLY:
+                               // leave the dialog open
+                               break;
 
-       switch (pos) {
-       case ImportAtEditCursor:
-               where = edit_cursor->current_frame;
-               break;
-       case ImportAtTimestamp:
-               where = -1;
-               break;
-       case ImportAtPlayhead:
-               where = playhead_cursor->current_frame;
-               break;
-       case ImportAtStart:
-               where = session->current_start_frame();
-               break;
+                       case RESPONSE_OK:
+                               sfbrowser->hide ();
+                               break;
+
+                       default:
+                               // cancel from the browser - we are done
+                               sfbrowser->hide ();
+                               return;
+               }
+
+               /* lets do it */
+
+               paths = sfbrowser->get_paths ();
+
+               ImportPosition pos = sfbrowser->get_position ();
+               ImportMode mode = sfbrowser->get_mode ();
+               ImportDisposition chns = sfbrowser->get_channel_disposition ();
+               nframes64_t where;
+
+               switch (pos) {
+                       case ImportAtEditPoint:
+                               where = get_preferred_edit_position ();
+                               break;
+                       case ImportAtTimestamp:
+                               where = -1;
+                               break;
+                       case ImportAtPlayhead:
+                               where = playhead_cursor->current_frame;
+                               break;
+                       case ImportAtStart:
+                               where = session->current_start_frame();
+                               break;
+               }
+
+               SrcQuality quality = sfbrowser->get_src_quality();
+
+
+               if (sfbrowser->copy_files_btn.get_active()) {
+                       do_import (paths, chns, mode, quality, where);
+               } else {
+                       do_embed (paths, chns, mode, where);
+               }
+
+               if (response == RESPONSE_APPLY) {
+                       sfbrowser->clear_selection ();
+                       keepRunning = true;
+               }
+
+       } while (keepRunning);
+}
+
+typedef std::map<PBD::ID,boost::shared_ptr<AudioSource> > AudioSourceList;
+
+/**
+ * Updating is still disabled, see note in libs/ardour/import.cc Session::import_audiofiles()
+ *
+ * all_or_nothing:
+ *   true  = show "Update", "Import" and "Skip"
+ *   false = show "Import", and "Cancel"
+ *
+ * Returns:
+ *     0  To update an existing source of the same name
+ *     1  To import/embed the file normally (make sure the new name will be unique)
+ *     2  If the user wants to skip this file
+ **/
+int
+Editor::check_whether_and_how_to_import(string path, bool all_or_nothing)
+{
+       string wave_name (Glib::path_get_basename(path));
+
+       AudioSourceList all_sources = session->get_audio_sources();
+       bool wave_name_exists = false;
+
+       for (AudioSourceList::iterator i = all_sources.begin(); i != all_sources.end(); ++i) {
+               boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(i->second);
+
+               string tmp (Glib::path_get_basename (afs->path()));
+
+               if (tmp == wave_name) {
+                       wave_name_exists = true;
+                       break;
+               }
        }
 
-       if (sfbrowser->import.get_active()) {
-               do_import (paths, chns, mode, where);
-       } else {
-               do_embed (paths, chns, mode, where);
+       int function = 1;
+
+
+       if (wave_name_exists) {
+               string message;
+               if (all_or_nothing) {
+                       // updating is still disabled
+                       //message = string_compose(_("The session already contains a source file named %1. Do you want to update that file (and thus all regions using the file) or import this file as a new file?"),wave_name);
+                       message = string_compose(_("The session already contains a source file named %1. This file will be imported as a new file, please confirm."),wave_name);
+               } else {
+                       message = string_compose(_("A source file %1 already exists. This operation will not update that source but import the file %2 as a new source, please confirm."), wave_name, wave_name);
+
+               }
+               MessageDialog dialog(message, false,Gtk::MESSAGE_QUESTION, Gtk::BUTTONS_NONE, true);
+
+               if (all_or_nothing) {
+                       // disabled
+                       //dialog.add_button("Update", 0);
+                       dialog.add_button("Import", 1);
+                       dialog.add_button("Skip",   2);
+               } else {
+                       dialog.add_button("Import", 1);
+                       dialog.add_button("Cancel", 2);
+               }
+               
+               
+               //dialog.add_button("Skip all", 4); // All or rest?
+
+               dialog.show();
+
+               function = dialog.run ();
+
+               dialog.hide();
        }
+
+       return function;
 }
 
 boost::shared_ptr<AudioTrack>
@@ -137,7 +258,12 @@ Editor::get_nth_selected_audio_track (int nth) const
        TrackSelection::iterator x;
        
        for (x = selection->tracks.begin(); nth > 0 && x != selection->tracks.end(); ++x) {
-               if (dynamic_cast<AudioTimeAxisView*>(*x)) {
+
+               atv = dynamic_cast<AudioTimeAxisView*>(*x);
+               
+               if (!atv) {
+                       continue;
+               } else if (atv->is_audio_track()) {
                        --nth;
                }
        }
@@ -148,91 +274,135 @@ Editor::get_nth_selected_audio_track (int nth) const
                atv = dynamic_cast<AudioTimeAxisView*>(*x);
        }
        
-       if (!atv) {
+       if (!atv || !atv->is_audio_track()) {
                return boost::shared_ptr<AudioTrack>();
        }
        
        return atv->audio_track();
 }      
 
+bool
+Editor::idle_do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mode, SrcQuality quality, nframes64_t& pos)
+{
+       _do_import (paths, chns, mode, quality, pos);
+       return false;
+}
+
+void
+Editor::do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mode, SrcQuality quality, nframes64_t& pos)
+{
+#ifdef GTKOSX
+       Glib::signal_idle().connect (bind (mem_fun (*this, &Editor::idle_do_import), paths, chns, mode, quality, pos));
+#else
+       _do_import (paths, chns, mode, quality, pos);
+#endif
+}
+
 void
-Editor::do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mode, nframes64_t& pos)
+Editor::_do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mode, SrcQuality quality, nframes64_t& pos)
 {
        boost::shared_ptr<AudioTrack> track;
        vector<ustring> to_import;
-       bool ok = false;
+       bool ok = true;
        int nth = 0;
 
        if (interthread_progress_window == 0) {
                build_interthread_progress_window ();
        }
 
-       switch (chns) {
-       case Editing::ImportDistinctFiles:
-               for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
+       if (chns == Editing::ImportMergeFiles) {
+               /* create 1 region from all paths, add to 1 track,
+                  ignore "track"
+               */
+               bool cancel = false;
+               for (vector<ustring>::iterator a = paths.begin(); a != paths.end() && ok; ++a) {
+                       int check = check_whether_and_how_to_import(*a, false);
+                       if (check == 2) {
+                               cancel = true;
+                               break;
+                       }
+               }
 
-                       to_import.clear ();
-                       to_import.push_back (*a);
+               if (!cancel) {
+                       if (import_sndfiles (paths, mode, quality, pos, 1, 1, track, false)) {
+                               ok = false;
+                       }
+               }
 
-                       if (mode == Editing::ImportToTrack) {
-                               track = get_nth_selected_audio_track (nth++);
+       } else {
+               bool replace;
+
+               for (vector<ustring>::iterator a = paths.begin(); a != paths.end() && ok; ++a) {
+
+                       int check = check_whether_and_how_to_import(*a, true);
+
+                       if (check == 2 ) { 
+                               // skip
+                               continue;
                        }
-                       
-                       if (import_sndfiles (to_import, mode, pos, 1, -1, track)) {
-                               goto out;
+
+                       if (check == 0) {
+                               fatal << "Updating existing sources should be disabled!" << endl;
+                               replace = true;
+                       } else if (check == 1) {
+                               replace = false;
                        }
+                       
 
-               }
-               break;
+                       switch (chns) {
+                               case Editing::ImportDistinctFiles:
 
-       case Editing::ImportDistinctChannels:
-               for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
+                                       to_import.clear ();
+                                       to_import.push_back (*a);
 
-                       to_import.clear ();
-                       to_import.push_back (*a);
+                                       if (mode == Editing::ImportToTrack) {
+                                               track = get_nth_selected_audio_track (nth++);
+                                       }
 
-                       if (import_sndfiles (to_import, mode, pos, -1, -1, track)) {
-                               goto out;
-                       }
+                                       if (import_sndfiles (to_import, mode, quality, pos, 1, -1, track, replace)) {
+                                               ok = false;
+                                       }
 
-               }
-               break;
+                                       break;
 
-       case Editing::ImportMergeFiles:
-               /* create 1 region from all paths, add to 1 track,
-                  ignore "track"
-               */
-               if (import_sndfiles (paths, mode, pos, 1, 1, track)) {
-                       goto out;
-               }
-               break;
+                               case Editing::ImportDistinctChannels:
 
-       case Editing::ImportSerializeFiles:
-               for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
+                                       to_import.clear ();
+                                       to_import.push_back (*a);
 
-                       to_import.clear ();
-                       to_import.push_back (*a);
-                       
-                       /* create 1 region from this path, add to 1 track,
-                          reuse "track" across paths
-                       */
+                                       if (import_sndfiles (to_import, mode, quality, pos, -1, -1, track, replace)) {
+                                               ok = false;
+                                       }
 
-                       if (import_sndfiles (to_import, mode, pos, 1, 1, track)) {
-                               goto out;
+                                       break;
+
+                               case Editing::ImportSerializeFiles:
+
+                                       to_import.clear ();
+                                       to_import.push_back (*a);
+
+                                       /* create 1 region from this path, add to 1 track,
+                                          reuse "track" across paths
+                                          */
+
+                                       if (import_sndfiles (to_import, mode, quality, pos, 1, 1, track, replace)) {
+                                               ok = false;
+                                       }
+
+                                       break;
+
+                               case Editing::ImportMergeFiles:
+                                       // Not entered
+                                       break;
                        }
+               }
 
+               if (ok) {
+                       session->save_state ("");
                }
-               break;
-       }
 
-       ok = true;
-       
-  out: 
-       if (ok) {
-               session->save_state ("");
+               interthread_progress_window->hide_all ();
        }
-
-       interthread_progress_window->hide_all ();
 }
 
 bool
@@ -319,14 +489,13 @@ Editor::_do_embed (vector<ustring> paths, ImportDisposition chns, ImportMode mod
 }
 
 int
-Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, nframes64_t& pos, 
-                        int target_regions, int target_tracks, boost::shared_ptr<AudioTrack>& track)
+Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality quality, nframes64_t& pos, 
+                        int target_regions, int target_tracks, boost::shared_ptr<AudioTrack>& track, bool replace)
 {
        WindowTitle title = string_compose (_("importing %1"), paths.front());
 
        interthread_progress_window->set_title (title.get_string());
        interthread_progress_window->set_position (Gtk::WIN_POS_MOUSE);
-       interthread_progress_window->show_all ();
        interthread_progress_bar.set_fraction (0.0f);
        interthread_cancel_label.set_text (_("Cancel Import"));
        current_interthread_info = &import_status;
@@ -336,11 +505,13 @@ Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, nframes64_t& po
        import_status.cancel = false;
        import_status.freeze = false;
        import_status.done = 0.0;
-       
+       import_status.quality = quality;
+       import_status.replace_existing_source = replace;
+
        interthread_progress_connection = Glib::signal_timeout().connect 
-               (bind (mem_fun(*this, &Editor::import_progress_timeout), (gpointer) 0), 100);
+               (bind (mem_fun(*this, &Editor::import_progress_timeout), (gpointer) 0), 500);
        
-       track_canvas.get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
+       track_canvas->get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
        ARDOUR_UI::instance()->flush_pending ();
 
        /* start import thread for this spec. this will ultimately call Session::import_audiofile()
@@ -367,12 +538,12 @@ Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, nframes64_t& po
                goto out;
        }
 
-       if (add_sources (paths, import_status.sources, pos, mode, target_regions, target_tracks, track) == 0) {
+       if (add_sources (paths, import_status.sources, pos, mode, target_regions, target_tracks, track, false) == 0) {
                session->save_state ("");
        }
 
   out:
-       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
        return 0;
 }
 
@@ -387,7 +558,7 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
        SoundFileInfo finfo;
        int ret = 0;
 
-       track_canvas.get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
+       track_canvas->get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
        ARDOUR_UI::instance()->flush_pending ();
 
        for (vector<Glib::ustring>::iterator p = paths.begin(); p != paths.end(); ++p) {
@@ -488,7 +659,7 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                        }
                }
                
-               track_canvas.get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
+               track_canvas->get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
 
                for (int n = 0; n < finfo.channels; ++n) {
                        try {
@@ -498,11 +669,15 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                                boost::shared_ptr<Source> s;
 
                                if ((s = session->source_by_path_and_channel (path, n)) == 0) {
+
+                                       cerr << "add embed/import source with defer_peaks = true\n";
+
                                        source = boost::dynamic_pointer_cast<AudioFileSource> (SourceFactory::createReadable 
                                                                                               (*session, path,  n,
                                                                                                (mode == ImportAsTapeTrack ? 
                                                                                                 AudioFileSource::Destructive : 
-                                                                                                AudioFileSource::Flag (0))));
+                                                                                                AudioFileSource::Flag (0)),
+                                                                                               true, true));
                                } else {
                                        source = boost::dynamic_pointer_cast<AudioFileSource> (s);
                                }
@@ -523,16 +698,16 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                goto out;
        }
 
-       ret = add_sources (paths, sources, pos, mode, target_regions, target_tracks, track);
+       ret = add_sources (paths, sources, pos, mode, target_regions, target_tracks, track, true);
 
   out:
-       track_canvas.get_window()->set_cursor (*current_canvas_cursor);
+       track_canvas->get_window()->set_cursor (*current_canvas_cursor);
        return ret;
 }
 
 int
 Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64_t& pos, ImportMode mode, 
-                    int target_regions, int target_tracks, boost::shared_ptr<AudioTrack>& track)
+                    int target_regions, int target_tracks, boost::shared_ptr<AudioTrack>& track, bool add_channel_suffix)
 {
        vector<boost::shared_ptr<AudioRegion> > regions;
        ustring region_name;
@@ -543,8 +718,7 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
                if (sources[0]->natural_position() != 0) {
                        pos = sources[0]->natural_position();
                } else {
-                       // XXX is this the best alternative ?
-                       pos = edit_cursor->current_frame;
+                       pos = get_preferred_edit_position ();
                }
        }
 
@@ -552,7 +726,7 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
 
                /* take all the sources we have and package them up as a region */
 
-               region_name = region_name_from_path (paths.front(), (sources.size() > 1));
+               region_name = region_name_from_path (paths.front(), (sources.size() > 1), false);
                
                regions.push_back (boost::dynamic_pointer_cast<AudioRegion> 
                                   (RegionFactory::create (sources, 0, sources[0]->length(), region_name, 0,
@@ -564,16 +738,17 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
 
                SourceList just_one;
                SourceList::iterator x;
+               uint32_t n;
 
-               for (x = sources.begin(); x != sources.end(); ++x) {
+               for (n = 0, x = sources.begin(); x != sources.end(); ++x, ++n) {
 
                        just_one.clear ();
                        just_one.push_back (*x);
-
+                       
                        boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource> (*x);
 
-                       region_name = region_name_from_path (afs->path(), false);
-                       
+                       region_name = region_name_from_path (afs->path(), false, true, sources.size(), n);
+
                        regions.push_back (boost::dynamic_pointer_cast<AudioRegion> 
                                           (RegionFactory::create (just_one, 0, (*x)->length(), region_name, 0,
                                                                   Region::Flag (Region::DefaultFlags|Region::WholeFile|Region::External))));
@@ -597,7 +772,9 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
                output_chan = input_chan;
        }
 
-       for (vector<boost::shared_ptr<AudioRegion> >::iterator r = regions.begin(); r != regions.end(); ++r) {
+       int n = 0;
+
+       for (vector<boost::shared_ptr<AudioRegion> >::iterator r = regions.begin(); r != regions.end(); ++r, ++n) {
 
                finish_bringing_in_audio (*r, input_chan, output_chan, pos, mode, track);
 
@@ -608,6 +785,12 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
                } 
        }
 
+       /* setup peak file building in another thread */
+
+       for (SourceList::iterator x = sources.begin(); x != sources.end(); ++x) {
+               SourceFactory::setup_peakfile (*x, true);
+       }
+
        return 0;
 }
        
@@ -615,7 +798,6 @@ int
 Editor::finish_bringing_in_audio (boost::shared_ptr<AudioRegion> region, uint32_t in_chans, uint32_t out_chans, nframes64_t& pos, 
                                  ImportMode mode, boost::shared_ptr<AudioTrack>& existing_track)
 {
-
        switch (mode) {
        case ImportAsRegion:
                /* relax, its been done */
@@ -625,17 +807,11 @@ Editor::finish_bringing_in_audio (boost::shared_ptr<AudioRegion> region, uint32_
        {
                if (!existing_track) {
 
-                       if (selection->tracks.empty()) {
-                               return -1;
-                       }
-                       
-                       AudioTimeAxisView* atv = dynamic_cast<AudioTimeAxisView*>(selection->tracks.front());
-                       
-                       if (!atv) {
+                       existing_track = get_nth_selected_audio_track (0);
+
+                       if (!existing_track) {
                                return -1;
                        }
-                       
-                       existing_track = atv->audio_track();
                }
 
                boost::shared_ptr<Playlist> playlist = existing_track->diskstream()->playlist();
@@ -652,12 +828,15 @@ Editor::finish_bringing_in_audio (boost::shared_ptr<AudioRegion> region, uint32_
        { 
                if (!existing_track) {
                        list<boost::shared_ptr<AudioTrack> > at (session->new_audio_track (in_chans, out_chans, Normal, 1));
+
                        if (at.empty()) {
                                return -1;
                        }
+
                        existing_track = at.front();
-                       existing_track->set_name (basename_nosuffix (region->name()), this);
+                       existing_track->set_name (region->name(), this);
                }
+
                boost::shared_ptr<AudioRegion> copy (boost::dynamic_pointer_cast<AudioRegion> (RegionFactory::create (region)));
                existing_track->diskstream()->playlist()->add_region (copy, pos);
                break;
@@ -691,7 +870,7 @@ Editor::_import_thread (void *arg)
 void *
 Editor::import_thread ()
 {
-       session->import_audiofile (import_status);
+       session->import_audiofiles (import_status);
        pthread_exit_pbd (0);
        /*NOTREACHED*/
        return 0;
@@ -700,6 +879,13 @@ Editor::import_thread ()
 gint
 Editor::import_progress_timeout (void *arg)
 {
+       bool reset = false;
+
+       if (!interthread_progress_window->is_visible()) {
+               interthread_progress_window->show_all ();
+               reset = true;
+       }
+       
        interthread_progress_label.set_text (import_status.doing_what);
 
        if (import_status.freeze) {
@@ -712,9 +898,20 @@ Editor::import_progress_timeout (void *arg)
                interthread_progress_bar.pulse ();
                return FALSE;
        } else {
-               interthread_progress_bar.set_fraction (import_status.progress);
+               float val = import_status.progress;
+               interthread_progress_bar.set_fraction (min (max (0.0f, val), 1.0f));
        }
 
-       return !(import_status.done || import_status.cancel);
+       if (reset) {
+
+               /* the window is now visible, speed up the updates */
+               
+               interthread_progress_connection.disconnect ();
+               interthread_progress_connection = Glib::signal_timeout().connect 
+                       (bind (mem_fun(*this, &Editor::import_progress_timeout), (gpointer) 0), 100);
+               return false;
+       } else {
+               return !(import_status.done || import_status.cancel);
+       }
 }