Move Diskstream ownership to Track, so that Session no longer holds lists of Diskstre...
[ardour.git] / gtk2_ardour / editor_audio_import.cc
index 7762a4232213efe31869c831360290a67c8ee7e3..119ea3bc5127f337887da759a03aa0646b8c1c15 100644 (file)
@@ -1,5 +1,5 @@
 /*
-    Copyright (C) 2000-2006 Paul Davis 
+    Copyright (C) 2000-2006 Paul Davis
 
     This program is free software; you can redistribute it and/or modify
     it under the terms of the GNU General Public License as published by
@@ -29,9 +29,9 @@
 #include "pbd/pthread_utils.h"
 #include "pbd/basename.h"
 #include "pbd/shortpath.h"
+#include "pbd/stateful_diff_command.h"
 
 #include <gtkmm2ext/choice.h>
-#include <gtkmm2ext/window_title.h>
 
 #include "ardour/session.h"
 #include "ardour/session_directory.h"
 #include "session_import_dialog.h"
 #include "utils.h"
 #include "gui_thread.h"
+#include "interthread_progress_window.h"
 
 #include "i18n.h"
 
 using namespace std;
 using namespace ARDOUR;
 using namespace PBD;
-using namespace sigc;
 using namespace Gtk;
 using namespace Gtkmm2ext;
 using namespace Editing;
@@ -75,14 +75,14 @@ using Glib::ustring;
 void
 Editor::add_external_audio_action (ImportMode mode_hint)
 {
-       if (session == 0) {
+       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 media"), session, 0, true, mode_hint);
+               sfbrowser = new SoundFileOmega (*this, _("Add Existing Media"), _session, 0, true, mode_hint);
        } else {
                sfbrowser->set_mode (mode_hint);
        }
@@ -96,17 +96,17 @@ Editor::external_audio_dialog ()
        vector<Glib::ustring> paths;
        uint32_t track_cnt;
 
-       if (session == 0) {
+       if (_session == 0) {
                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()) {
@@ -115,7 +115,7 @@ Editor::external_audio_dialog ()
        }
 
        if (sfbrowser == 0) {
-               sfbrowser = new SoundFileOmega (*this, _("Add existing media"), session, track_cnt, true);
+               sfbrowser = new SoundFileOmega (*this, _("Add Existing Media"), _session, track_cnt, true);
        } else {
                sfbrowser->reset (track_cnt);
        }
@@ -165,7 +165,7 @@ Editor::external_audio_dialog ()
                                where = playhead_cursor->current_frame;
                                break;
                        case ImportAtStart:
-                               where = session->current_start_frame();
+                               where = _session->current_start_frame();
                                break;
                }
 
@@ -189,7 +189,7 @@ Editor::external_audio_dialog ()
 void
 Editor::session_import_dialog ()
 {
-       SessionImportDialog dialog (*session);
+       SessionImportDialog dialog (_session);
        ensure_float (dialog);
        dialog.run ();
 }
@@ -213,7 +213,7 @@ Editor::check_whether_and_how_to_import(string path, bool all_or_nothing)
 {
        string wave_name (Glib::path_get_basename(path));
 
-       SourceMap all_sources = session->get_sources();
+       SourceMap all_sources = _session->get_sources();
        bool wave_name_exists = false;
 
        for (SourceMap::iterator i = all_sources.begin(); i != all_sources.end(); ++i) {
@@ -231,9 +231,9 @@ Editor::check_whether_and_how_to_import(string path, bool all_or_nothing)
                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);
+                       message = string_compose (_("The session already contains a source file named %1.  Do you want to import %1 as a new file, or skip it?"), 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);
+                       message = string_compose (_("The session already contains a source file named %1.  Do you want to import %2 as a new source, or skip it?"), wave_name, wave_name);
 
                }
                MessageDialog dialog(message, false, Gtk::MESSAGE_QUESTION, Gtk::BUTTONS_NONE, true);
@@ -247,7 +247,7 @@ Editor::check_whether_and_how_to_import(string path, bool all_or_nothing)
                        dialog.add_button("Import", 1);
                        dialog.add_button("Cancel", 2);
                }
-               
+
                //dialog.add_button("Skip all", 4); // All or rest?
 
                dialog.show();
@@ -265,60 +265,60 @@ Editor::get_nth_selected_audio_track (int nth) const
 {
        AudioTimeAxisView* atv;
        TrackSelection::iterator x;
-       
+
        for (x = selection->tracks.begin(); nth > 0 && x != selection->tracks.end(); ++x) {
 
                atv = dynamic_cast<AudioTimeAxisView*>(*x);
-               
+
                if (!atv) {
                        continue;
                } else if (atv->is_audio_track()) {
                        --nth;
                }
        }
-       
+
        if (x == selection->tracks.end()) {
                atv = dynamic_cast<AudioTimeAxisView*>(selection->tracks.back());
        } else {
                atv = dynamic_cast<AudioTimeAxisView*>(*x);
        }
-       
+
        if (!atv || !atv->is_audio_track()) {
                return boost::shared_ptr<AudioTrack>();
        }
-       
+
        return atv->audio_track();
-}      
+}
 
 boost::shared_ptr<MidiTrack>
 Editor::get_nth_selected_midi_track (int nth) const
 {
        MidiTimeAxisView* mtv;
        TrackSelection::iterator x;
-       
+
        for (x = selection->tracks.begin(); nth > 0 && x != selection->tracks.end(); ++x) {
 
                mtv = dynamic_cast<MidiTimeAxisView*>(*x);
-               
+
                if (!mtv) {
                        continue;
                } else if (mtv->is_midi_track()) {
                        --nth;
                }
        }
-       
+
        if (x == selection->tracks.end()) {
                mtv = dynamic_cast<MidiTimeAxisView*>(selection->tracks.back());
        } else {
                mtv = dynamic_cast<MidiTimeAxisView*>(*x);
        }
-       
+
        if (!mtv || !mtv->is_midi_track()) {
                return boost::shared_ptr<MidiTrack>();
        }
-       
+
        return mtv->midi_track();
-}      
+}
 
 void
 Editor::do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mode, SrcQuality quality, nframes64_t& pos)
@@ -327,9 +327,11 @@ Editor::do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mod
        vector<ustring> to_import;
        int nth = 0;
 
-       if (interthread_progress_window == 0) {
-               build_interthread_progress_window ();
-       }
+       current_interthread_info = &import_status;
+       import_status.current = 1;
+       import_status.total = paths.size ();
+
+       ImportProgressWindow ipw (&import_status, _("Import"), _("Cancel Import"));
 
        if (chns == Editing::ImportMergeFiles) {
 
@@ -347,70 +349,69 @@ Editor::do_import (vector<ustring> paths, ImportDisposition chns, ImportMode mod
                }
 
                if (!cancel) {
-                       import_sndfiles (paths, mode, quality, pos, 1, 1, track, false, paths.size());
+                       import_sndfiles (paths, mode, quality, pos, 1, 1, track, false);
                }
 
        } else {
 
-               bool replace = false;
+                bool replace = false;
                bool ok = true;
-               vector<ustring>::size_type total = paths.size();
-
-               for (vector<ustring>::iterator a = paths.begin(); a != paths.end() && ok; ++a) {
 
-                       int check = check_whether_and_how_to_import(*a, true);
+               for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
 
-                       if (check == 2 ) { 
+                        const int check = check_whether_and_how_to_import (*a, true);
+                        
+                       switch (check) {
+                       case 2:
                                // user said skip
                                continue;
-                       }
-
-                       if (check == 0) {
-                               fatal << "Updating existing sources should be disabled!" << endl;
-                               replace = true;
-                       } else if (check == 1) {
+                       case 0:
+                               fatal << "Updating existing sources should be disabled!" << endmsg;
+                               /* NOTREACHED*/
+                               break;
+                       case 1:
                                replace = false;
+                               break;
+                       default:
+                               fatal << "Illegal return " << check <<  " from check_whether_and_how_to_import()!" << endmsg;
+                               /* NOTREACHED*/
                        }
-                       
-
 
-                       switch (chns) {
-                       case Editing::ImportDistinctFiles:
-                               
-                               to_import.clear ();
-                               to_import.push_back (*a);
-                               
-                               if (mode == Editing::ImportToTrack) {
-                                       track = get_nth_selected_audio_track (nth++);
-                               }
+                        switch (chns) {
+                        case Editing::ImportDistinctFiles:
+                                
+                                to_import.clear ();
+                                to_import.push_back (*a);
+                                
+                                if (mode == Editing::ImportToTrack) {
+                                        track = get_nth_selected_audio_track (nth++);
+                                }
+                       
+                                ok = (import_sndfiles (to_import, mode, quality, pos, 1, -1, track, replace) == 0);
+                                break;
                                
-                               ok = (import_sndfiles (to_import, mode, quality, pos, 1, -1, track, replace, total) == 0);
-                               break;
+                        case Editing::ImportDistinctChannels:
                                
-                       case Editing::ImportDistinctChannels:
+                                to_import.clear ();
+                                to_import.push_back (*a);
                                
-                               to_import.clear ();
-                               to_import.push_back (*a);
+                                ok = (import_sndfiles (to_import, mode, quality, pos, -1, -1, track, replace) == 0);
+                                break;
                                
-                               ok = (import_sndfiles (to_import, mode, quality, pos, -1, -1, track, replace, total) == 0);
-                               break;
+                        case Editing::ImportSerializeFiles:
                                
-                       case Editing::ImportSerializeFiles:
-                               
-                               to_import.clear ();
-                               to_import.push_back (*a);
+                                to_import.clear ();
+                                to_import.push_back (*a);
 
-                               ok = (import_sndfiles (to_import, mode, quality, pos, 1, 1, track, replace, total) == 0);
-                               break;
+                                ok = (import_sndfiles (to_import, mode, quality, pos, 1, 1, track, replace) == 0);
+                                break;
 
-                       case Editing::ImportMergeFiles:
-                               // Not entered, handled in earlier if() branch
-                               break;
-                       }
-               }
+                        case Editing::ImportMergeFiles:
+                                // Not entered, handled in earlier if() branch
+                                break;
+                        }
+                }
        }
-
-       interthread_progress_window->hide_all ();
 }
 
 void
@@ -439,7 +440,7 @@ Editor::do_embed (vector<ustring> paths, ImportDisposition chns, ImportMode mode
                        }
                }
                break;
-               
+
        case Editing::ImportDistinctChannels:
                for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
 
@@ -456,7 +457,7 @@ Editor::do_embed (vector<ustring> paths, ImportDisposition chns, ImportMode mode
                if (embed_sndfiles (paths, multi, check_sample_rate, mode, pos, 1, 1, track) < -1) {
                        goto out;
                }
-       break;
+                break;
 
        case Editing::ImportSerializeFiles:
                for (vector<ustring>::iterator a = paths.begin(); a != paths.end(); ++a) {
@@ -472,25 +473,17 @@ Editor::do_embed (vector<ustring> paths, ImportDisposition chns, ImportMode mode
        }
 
        ok = true;
-       
-  out: 
+
+  out:
        if (ok) {
-               session->save_state ("");
+               _session->save_state ("");
        }
 }
 
 int
-Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality quality, nframes64_t& pos, 
-                        int target_regions, int target_tracks, boost::shared_ptr<Track> track, bool replace, uint32_t total)
+Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality quality, nframes64_t& pos,
+                        int target_regions, int target_tracks, boost::shared_ptr<Track>& 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_bar.set_fraction (0.0f);
-       interthread_cancel_label.set_text (_("Cancel Import"));
-       current_interthread_info = &import_status;
-
        import_status.paths = paths;
        import_status.done = false;
        import_status.cancel = false;
@@ -498,7 +491,6 @@ Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality qual
        import_status.done = 0.0;
        import_status.quality = quality;
        import_status.replace_existing_source = replace;
-       import_status.total = total;
 
        import_status.mode = mode;
        import_status.pos = pos;
@@ -506,9 +498,7 @@ Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality qual
        import_status.target_regions = target_regions;
        import_status.track = track;
        import_status.replace = replace;
-       interthread_progress_connection = Glib::signal_timeout().connect 
-               (bind (mem_fun(*this, &Editor::import_progress_timeout), (gpointer) 0), 500);
-       
+
        track_canvas->get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
        gdk_flush ();
 
@@ -517,30 +507,28 @@ Editor::import_sndfiles (vector<ustring> paths, ImportMode mode, SrcQuality qual
           (the GUI) to direct additional steps after that.
        */
 
-       pthread_create_and_store ("import", &import_status.thread, 0, _import_thread, this);
+       pthread_create_and_store ("import", &import_status.thread, _import_thread, this);
        pthread_detach (import_status.thread);
-       
+
        while (!import_status.done && !import_status.cancel) {
                gtk_main_iteration ();
        }
 
-       interthread_progress_window->hide ();
        import_status.done = true;
-       interthread_progress_connection.disconnect ();
-       
+
        if (!import_status.cancel && !import_status.sources.empty()) {
-               if (add_sources (import_status.paths, 
-                                import_status.sources, 
-                                import_status.pos, 
-                                import_status.mode, 
-                                import_status.target_regions, 
-                                import_status.target_tracks, 
-                                import_status.track, false) == 0) {
-                       session->save_state ("");
+               if (add_sources (import_status.paths,
+                                import_status.sources,
+                                import_status.pos,
+                                import_status.mode,
+                                import_status.target_regions,
+                                import_status.target_tracks,
+                                track, false) == 0) {
+                       _session->save_state ("");
                }
-               
+
                /* update position from results */
-               
+
                pos = import_status.pos;
        }
 
@@ -568,64 +556,69 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
 
                ustring path = *p;
 
-               /* lets see if we can link it into the session */
-               
-               sys::path tmp = session->session_directory().sound_path() / Glib::path_get_basename(path);
-               linked_path = tmp.to_string();
+               if (Config->get_try_link_for_embed()) {
 
-               path_to_use = linked_path;
-               
-               if (link (path.c_str(), linked_path.c_str()) == 0) {
-
-                       /* there are many reasons why link(2) might have failed.
-                          but if it succeeds, we now have a link in the
-                          session sound dir that will protect against
-                          unlinking of the original path. nice.
-                       */
+                       /* lets see if we can link it into the session */
                        
-                       path = linked_path;
-                       path_to_use = Glib::path_get_basename (path);
-
-               } else {
-
-                       /* one possible reason is that its already linked */
-
-                       if (errno == EEXIST) {
-                               struct stat sb;
+                       sys::path tmp = _session->session_directory().sound_path() / Glib::path_get_basename(path);
+                       linked_path = tmp.to_string();
+                       
+                       path_to_use = linked_path;
+                       
+                       if (link (path.c_str(), linked_path.c_str()) == 0) {
+                               
+                               /* there are many reasons why link(2) might have failed.
+                                  but if it succeeds, we now have a link in the
+                                  session sound dir that will protect against
+                                  unlinking of the original path. nice.
+                               */
+                               
+                               path = linked_path;
+                               path_to_use = Glib::path_get_basename (path);
+                               
+                       } else {
 
-                               if (stat (linked_path.c_str(), &sb) == 0) {
-                                       if (sb.st_nlink > 1) { // its a hard link, assume its the one we want
-                                               path = linked_path;
-                                               path_to_use = Glib::path_get_basename (path);
+                               /* one possible reason is that its already linked */
+                               
+                               if (errno == EEXIST) {
+                                       struct stat sb;
+                                       
+                                       if (stat (linked_path.c_str(), &sb) == 0) {
+                                               if (sb.st_nlink > 1) { // its a hard link, assume its the one we want
+                                                       path = linked_path;
+                                                       path_to_use = Glib::path_get_basename (path);
+                                               }
                                        }
                                }
                        }
                }
-               
+
                /* note that we temporarily truncated _id at the colon */
-               
+
                string error_msg;
 
                if (!AudioFileSource::get_soundfile_info (path, finfo, error_msg)) {
                        error << string_compose(_("Editor: cannot open file \"%1\", (%2)"), path, error_msg ) << endmsg;
                        goto out;
                }
-               
-               if (check_sample_rate  && (finfo.samplerate != (int) session->frame_rate())) {
+
+               if (check_sample_rate  && (finfo.samplerate != (int) _session->frame_rate())) {
                        vector<string> choices;
-                       
+
                        if (multifile) {
                                choices.push_back (_("Cancel entire import"));
                                choices.push_back (_("Don't embed it"));
                                choices.push_back (_("Embed all without questions"));
-                       
+
                                Gtkmm2ext::Choice rate_choice (
-                                       string_compose (_("%1\nThis audiofile's sample rate doesn't match the session sample rate!"), 
+                                       _("Sample rate"),
+                                       string_compose (_("%1\nThis audiofile's sample rate doesn't match the session sample rate!"),
                                                        short_path (path, 40)),
-                                       choices, false);
-                               
+                                       choices, false
+                                       );
+
                                int resx = rate_choice.run ();
-                               
+
                                switch (resx) {
                                case 0: /* stop a multi-file import */
                                        ret = -2;
@@ -645,13 +638,15 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                        } else {
                                choices.push_back (_("Cancel"));
                                choices.push_back (_("Embed it anyway"));
-                       
+
                                Gtkmm2ext::Choice rate_choice (
+                                       _("Sample rate"),
                                        string_compose (_("%1\nThis audiofile's sample rate doesn't match the session sample rate!"), path),
-                                       choices, false);
-                               
+                                       choices, false
+                                       );
+
                                int resx = rate_choice.run ();
-                               
+
                                switch (resx) {
                                case 0: /* don't import */
                                        ret = -1;
@@ -664,7 +659,7 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                                }
                        }
                }
-               
+
                track_canvas->get_window()->set_cursor (Gdk::Cursor (Gdk::WATCH));
 
                for (int n = 0; n < finfo.channels; ++n) {
@@ -674,12 +669,12 @@ 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) {
+                               if ((s = _session->source_by_path_and_channel (path, n)) == 0) {
 
                                        source = boost::dynamic_pointer_cast<AudioFileSource> (
-                                                       SourceFactory::createReadable (DataType::AUDIO, *session,
-                                                                       path_to_use, false, n,
-                                                                       (mode == ImportAsTapeTrack
+                                               SourceFactory::createReadable (DataType::AUDIO, *_session,
+                                                                              path_to_use, n,
+                                                                              (mode == ImportAsTapeTrack
                                                                                ? Source::Destructive
                                                                                : Source::Flag (0)),
                                                                        true, true));
@@ -688,13 +683,13 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
                                }
 
                                sources.push_back(source);
-                       } 
-                       
+                       }
+
                        catch (failed_constructor& err) {
                                error << string_compose(_("could not open %1"), path) << endmsg;
                                goto out;
                        }
-                       
+
                        ARDOUR_UI::instance()->flush_pending ();
                }
        }
@@ -711,19 +706,22 @@ Editor::embed_sndfiles (vector<Glib::ustring> paths, bool multifile,
 }
 
 int
-Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64_t& pos, ImportMode mode, 
-                    int target_regions, int target_tracks, boost::shared_ptr<Track>& track, bool add_channel_suffix)
+Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64_t& pos, ImportMode mode,
+                    int target_regions, int target_tracks, boost::shared_ptr<Track>& track, bool /*add_channel_suffix*/)
 {
        vector<boost::shared_ptr<Region> > regions;
        ustring region_name;
        uint32_t input_chan = 0;
        uint32_t output_chan = 0;
+       bool use_timestamp;
 
-       if (pos == -1) { // "use timestamp"
+       use_timestamp = (pos == -1);
+
+       if (use_timestamp) {
                if (sources[0]->natural_position() != 0) {
                        pos = sources[0]->natural_position();
                } else {
-                       pos = get_preferred_edit_position ();
+                       pos = _session->current_start_frame();
                }
        }
 
@@ -736,10 +734,25 @@ 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), false);
+
+               PropertyList plist; 
                
-               regions.push_back (RegionFactory::create (sources, 0, sources[0]->length(pos), region_name, 0,
-                               Region::Flag (Region::DefaultFlags|Region::WholeFile|Region::External)));
-               
+               plist.add (ARDOUR::Properties::start, 0);
+               plist.add (ARDOUR::Properties::length, sources[0]->length (pos));
+               plist.add (ARDOUR::Properties::name, region_name);
+               plist.add (ARDOUR::Properties::layer, 0);
+               plist.add (ARDOUR::Properties::whole_file, true);
+               plist.add (ARDOUR::Properties::external, true);
+
+               boost::shared_ptr<Region> r = RegionFactory::create (sources, plist);
+
+               if (use_timestamp && boost::dynamic_pointer_cast<AudioRegion>(r)) {
+                       boost::dynamic_pointer_cast<AudioRegion>(r)->special_set_position(sources[0]->natural_position());
+               }
+
+               regions.push_back (r);
+
+
        } else if (target_regions == -1 || target_regions > 1) {
 
                /* take each source and create a region for each one */
@@ -752,12 +765,25 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
 
                        just_one.clear ();
                        just_one.push_back (*x);
-                       
+
                        region_name = region_name_from_path ((*x)->path(), false, false, sources.size(), n);
 
-                       regions.push_back (RegionFactory::create (just_one, 0, (*x)->length(pos), region_name, 0,
-                                       Region::Flag (Region::DefaultFlags|Region::WholeFile|Region::External)));
+                       PropertyList plist; 
+                       
+                       plist.add (ARDOUR::Properties::start, 0);
+                       plist.add (ARDOUR::Properties::length, (*x)->length (pos));
+                       plist.add (ARDOUR::Properties::name, region_name);
+                       plist.add (ARDOUR::Properties::layer, 0);
+                       plist.add (ARDOUR::Properties::whole_file, true);
+                       plist.add (ARDOUR::Properties::external, true);
+
+                       boost::shared_ptr<Region> r = RegionFactory::create (just_one, plist);
 
+                       if (use_timestamp && boost::dynamic_pointer_cast<AudioRegion>(r)) {
+                               boost::dynamic_pointer_cast<AudioRegion>(r)->special_set_position((*x)->natural_position());
+                       }
+
+                       regions.push_back (r);
                }
        }
 
@@ -772,7 +798,7 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
        }
 
        if (Config->get_output_auto_connect() & AutoConnectMaster) {
-               output_chan = (session->master_out() ? session->master_out()->n_inputs().n_audio() : input_chan);
+               output_chan = (_session->master_out() ? _session->master_out()->n_inputs().n_audio() : input_chan);
        } else {
                output_chan = input_chan;
        }
@@ -787,7 +813,7 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
                        track.reset ();
                } else {
                        pos += (*r)->length();
-               } 
+               }
        }
 
        /* setup peak file building in another thread */
@@ -798,9 +824,9 @@ Editor::add_sources (vector<Glib::ustring> paths, SourceList& sources, nframes64
 
        return 0;
 }
-       
+
 int
-Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t in_chans, uint32_t out_chans, nframes64_t& pos, 
+Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t in_chans, uint32_t out_chans, nframes64_t& pos,
                                  ImportMode mode, boost::shared_ptr<Track>& existing_track)
 {
        boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion>(region);
@@ -810,7 +836,7 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
        case ImportAsRegion:
                /* relax, its been done */
                break;
-               
+
        case ImportToTrack:
        {
                if (!existing_track) {
@@ -826,21 +852,21 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
                        }
                }
 
-               boost::shared_ptr<Playlist> playlist = existing_track->diskstream()->playlist();
-               boost::shared_ptr<Region> copy (RegionFactory::create (region));
+               boost::shared_ptr<Playlist> playlist = existing_track->playlist();
+               boost::shared_ptr<Region> copy (RegionFactory::create (region, region->properties()));
                begin_reversible_command (_("insert file"));
-               XMLNode &before = playlist->get_state();
+                playlist->clear_history ();
                playlist->add_region (copy, pos);
-               session->add_command (new MementoCommand<Playlist>(*playlist, &before, &playlist->get_state()));
+               _session->add_command (new StatefulDiffCommand (playlist));
                commit_reversible_command ();
                break;
        }
 
        case ImportAsTrack:
-       { 
+       {
                if (!existing_track) {
                        if (ar) {
-                               list<boost::shared_ptr<AudioTrack> > at (session->new_audio_track (in_chans, out_chans, Normal, 0, 1));
+                               list<boost::shared_ptr<AudioTrack> > at (_session->new_audio_track (in_chans, out_chans, Normal, 0, 1));
 
                                if (at.empty()) {
                                        return -1;
@@ -848,7 +874,7 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
 
                                existing_track = at.front();
                        } else if (mr) {
-                               list<boost::shared_ptr<MidiTrack> > mt (session->new_midi_track (Normal, 0, 1));
+                               list<boost::shared_ptr<MidiTrack> > mt (_session->new_midi_track (Normal, 0, 1));
 
                                if (mt.empty()) {
                                        return -1;
@@ -861,7 +887,7 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
                }
 
                boost::shared_ptr<Region> copy (RegionFactory::create (region));
-               existing_track->diskstream()->playlist()->add_region (copy, pos);
+               existing_track->playlist()->add_region (copy, pos);
                break;
        }
 
@@ -871,11 +897,11 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
                if (!ar)
                        return -1;
 
-               list<boost::shared_ptr<AudioTrack> > at (session->new_audio_track (in_chans, out_chans, Destructive));
+               list<boost::shared_ptr<AudioTrack> > at (_session->new_audio_track (in_chans, out_chans, Destructive));
                if (!at.empty()) {
                        boost::shared_ptr<Region> copy (RegionFactory::create (region));
                        at.front()->set_name (basename_nosuffix (copy->name()));
-                       at.front()->diskstream()->playlist()->add_region (copy, pos);
+                       at.front()->playlist()->add_region (copy, pos);
                }
                break;
        }
@@ -887,7 +913,7 @@ Editor::finish_bringing_in_material (boost::shared_ptr<Region> region, uint32_t
 void *
 Editor::_import_thread (void *arg)
 {
-       PBD::notify_gui_about_thread_creation (pthread_self(), X_("Import"));
+       SessionEvent::create_per_thread_pool ("import events", 64);
 
        Editor *ed = (Editor *) arg;
        return ed->import_thread ();
@@ -896,48 +922,8 @@ Editor::_import_thread (void *arg)
 void *
 Editor::import_thread ()
 {
-       session->import_audiofiles (import_status);
+       _session->import_audiofiles (import_status);
        pthread_exit_pbd (0);
        /*NOTREACHED*/
        return 0;
 }
-
-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) {
-               interthread_cancel_button.set_sensitive(false);
-       } else {
-               interthread_cancel_button.set_sensitive(true);
-       }
-
-       if (import_status.doing_what == "building peak files") {
-               interthread_progress_bar.pulse ();
-               return FALSE;
-       } else {
-               float val = import_status.progress;
-               interthread_progress_bar.set_fraction (min (max (0.0f, val), 1.0f));
-       }
-
-       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);
-       }
-}
-