ensure that imported regions have names based on the files they are based on

[ardour.git] / libs / ardour / audiofilesource.cc

diff --git a/libs/ardour/audiofilesource.cc b/libs/ardour/audiofilesource.cc
index 8de786eb0e0901f94c64cf3d516a328651e0772d..297c02e418dba264c2795be1a9abe808810302e8 100644 (file)
--- a/libs/ardour/audiofilesource.cc
+++ b/libs/ardour/audiofilesource.cc
@@ -45,6 +45,7 @@
 #include <glibmm/thread.h>
 
 #include "ardour/audiofilesource.h"
+#include "ardour/debug.h"
 #include "ardour/sndfile_helpers.h"
 #include "ardour/sndfilesource.h"
 #include "ardour/session.h"
@@ -69,7 +70,7 @@ using namespace Glib;
 
 ustring AudioFileSource::peak_dir = "";
 
-sigc::signal<void> AudioFileSource::HeaderPositionOffsetChanged;
+PBD::Signal0<void> AudioFileSource::HeaderPositionOffsetChanged;
 uint64_t           AudioFileSource::header_position_offset = 0;
 
 /* XXX maybe this too */
@@ -91,31 +92,30 @@ struct SizedSampleBuffer {
 Glib::StaticPrivate<SizedSampleBuffer> thread_interleave_buffer = GLIBMM_STATIC_PRIVATE_INIT;
 
 /** Constructor used for existing internal-to-session files. */
-AudioFileSource::AudioFileSource (Session& s, const ustring& path, bool embedded, Source::Flag flags)
+AudioFileSource::AudioFileSource (Session& s, const ustring& path, Source::Flag flags)
        : Source (s, DataType::AUDIO, path, flags)
        , AudioSource (s, path)
-       , FileSource (s, DataType::AUDIO, path, embedded, flags)
+       , FileSource (s, DataType::AUDIO, path, flags)
 {
-       if (init (path, true)) {
+       if (init (_path, true)) {
                throw failed_constructor ();
        }
+
 }
 
 /** Constructor used for new internal-to-session files. */
-AudioFileSource::AudioFileSource (Session& s, const ustring& path, bool embedded, Source::Flag flags,
+AudioFileSource::AudioFileSource (Session& s, const ustring& path, Source::Flag flags,
                                  SampleFormat /*samp_format*/, HeaderFormat /*hdr_format*/)
        : Source (s, DataType::AUDIO, path, flags)
        , AudioSource (s, path)
-       , FileSource (s, DataType::AUDIO, path, embedded, flags)
+       , FileSource (s, DataType::AUDIO, path, flags)
 {
-       _is_embedded = false;
-
-       if (init (path, false)) {
+       if (init (_path, false)) {
                throw failed_constructor ();
        }
 }
 
-/** Constructor used for existing internal-to-session files.  File must exist. */
+/** Constructor used for existing internal-to-session files via XML.  File must exist. */
 AudioFileSource::AudioFileSource (Session& s, const XMLNode& node, bool must_exist)
        : Source (s, node)
        , AudioSource (s, node)
@@ -124,14 +124,15 @@ AudioFileSource::AudioFileSource (Session& s, const XMLNode& node, bool must_exi
        if (set_state (node, Stateful::loading_state_version)) {
                throw failed_constructor ();
        }
-
-       if (init (_name, must_exist)) {
+       
+       if (init (_path, must_exist)) {
                throw failed_constructor ();
        }
 }
 
 AudioFileSource::~AudioFileSource ()
 {
+       DEBUG_TRACE (DEBUG::Destruction, string_compose ("AudioFileSource destructor %1, removable? %2\n", _path, removable()));
        if (removable()) {
                unlink (_path.c_str());
                unlink (peakpath.c_str());
@@ -168,7 +169,7 @@ AudioFileSource::find_broken_peakfile (ustring peak_path, ustring audio_path)
 
        if (Glib::file_test (str, Glib::FILE_TEST_EXISTS)) {
 
-               if (is_embedded()) {
+               if (!within_session()) {
 
                        /* it would be nice to rename it but the nature of
                           the bug means that we can't reliably use it.