diff options
author | Paul Davis <paul@linuxaudiosystems.com> | 2020-03-23 21:47:13 -0600 |
---|---|---|
committer | Paul Davis <paul@linuxaudiosystems.com> | 2020-03-23 21:47:13 -0600 |
commit | 0f63b829439e74c9ca8e3e1b8597651ddff361c6 (patch) | |
tree | 52ff6ab429c3cc0ea746a38864ee04dbd8fac9a0 /libs | |
parent | 3bbad66a999dbe402209881de072720d78f01793 (diff) |
fix error in multiple calls to SourceFactory::createWritable()
removal of tape tracks removed an intermediate argument in the argument list; presence of default args for the
last two arguments and implicit conversion from int->bool prevented the compiler from complaining
about any existing calls.
This supplements/extends a54b000a70
Diffstat (limited to 'libs')
-rw-r--r-- | libs/ardour/filter.cc | 2 | ||||
-rw-r--r-- | libs/ardour/import.cc | 4 | ||||
-rw-r--r-- | libs/ardour/lua_api.cc | 7 | ||||
-rw-r--r-- | libs/ardour/midi_region.cc | 7 | ||||
-rw-r--r-- | libs/ardour/session.cc | 13 | ||||
-rw-r--r-- | libs/ardour/test/audio_region_test.cc | 2 |
6 files changed, 12 insertions, 23 deletions
diff --git a/libs/ardour/filter.cc b/libs/ardour/filter.cc index c5a0ff4282..040a1cab83 100644 --- a/libs/ardour/filter.cc +++ b/libs/ardour/filter.cc @@ -86,7 +86,7 @@ Filter::make_new_sources (boost::shared_ptr<Region> region, SourceList& nsrcs, s nsrcs.push_back (boost::dynamic_pointer_cast<Source> ( SourceFactory::createWritable (region->data_type(), session, - path, false, sample_rate))); + path, sample_rate))); } catch (failed_constructor& err) { diff --git a/libs/ardour/import.cc b/libs/ardour/import.cc index f99519bd05..30640d3627 100644 --- a/libs/ardour/import.cc +++ b/libs/ardour/import.cc @@ -202,9 +202,7 @@ create_mono_sources_for_writing (const vector<string>& new_paths, try { const DataType type = SMFSource::safe_midi_file_extension (*i) ? DataType::MIDI : DataType::AUDIO; - source = SourceFactory::createWritable (type, sess, - i->c_str(), - samplerate); + source = SourceFactory::createWritable (type, sess, i->c_str(), samplerate); } catch (const failed_constructor& err) { diff --git a/libs/ardour/lua_api.cc b/libs/ardour/lua_api.cc index 1a398c6fa2..b446c56355 100644 --- a/libs/ardour/lua_api.cc +++ b/libs/ardour/lua_api.cc @@ -1055,10 +1055,9 @@ LuaAPI::Rubberband::process (luabridge::LuaRef cb) return rv; } try { - _asrc.push_back (boost::dynamic_pointer_cast<AudioSource> ( - SourceFactory::createWritable ( - DataType::AUDIO, session, - path, false, sample_rate))); + + _asrc.push_back (boost::dynamic_pointer_cast<AudioSource> (SourceFactory::createWritable (DataType::AUDIO, session, path, sample_rate))); + } catch (failed_constructor& err) { cleanup (true); return rv; diff --git a/libs/ardour/midi_region.cc b/libs/ardour/midi_region.cc index 80da942589..5d6c774cc3 100644 --- a/libs/ardour/midi_region.cc +++ b/libs/ardour/midi_region.cc @@ -145,9 +145,7 @@ MidiRegion::do_export (string path) const /* caller must check for pre-existing file */ assert (!path.empty()); assert (!Glib::file_test (path, Glib::FILE_TEST_EXISTS)); - newsrc = boost::dynamic_pointer_cast<MidiSource>( - SourceFactory::createWritable(DataType::MIDI, _session, - path, false, _session.sample_rate())); + newsrc = boost::dynamic_pointer_cast<MidiSource>(SourceFactory::createWritable(DataType::MIDI, _session, path, _session.sample_rate())); BeatsSamplesConverter bfc (_session.tempo_map(), _position); Temporal::Beats const bbegin = bfc.from (_start); @@ -177,8 +175,7 @@ MidiRegion::clone (string path) const assert (!path.empty()); assert (!Glib::file_test (path, Glib::FILE_TEST_EXISTS)); newsrc = boost::dynamic_pointer_cast<MidiSource>( - SourceFactory::createWritable(DataType::MIDI, _session, - path, false, _session.sample_rate())); + SourceFactory::createWritable(DataType::MIDI, _session, path, _session.sample_rate())); return clone (newsrc); } diff --git a/libs/ardour/session.cc b/libs/ardour/session.cc index 860ff846aa..9e4829666f 100644 --- a/libs/ardour/session.cc +++ b/libs/ardour/session.cc @@ -4907,8 +4907,7 @@ Session::create_audio_source_for_session (size_t n_chans, string const & base, u const string path = new_audio_source_path (base, n_chans, chan, true); if (!path.empty()) { - return boost::dynamic_pointer_cast<AudioFileSource> ( - SourceFactory::createWritable (DataType::AUDIO, *this, path, sample_rate(), true, true)); + return boost::dynamic_pointer_cast<AudioFileSource> (SourceFactory::createWritable (DataType::AUDIO, *this, path, sample_rate(), true, true)); } else { throw failed_constructor (); } @@ -4921,9 +4920,7 @@ Session::create_midi_source_for_session (string const & basic_name) const string path = new_midi_source_path (basic_name); if (!path.empty()) { - return boost::dynamic_pointer_cast<SMFSource> ( - SourceFactory::createWritable ( - DataType::MIDI, *this, path, false, sample_rate())); + return boost::dynamic_pointer_cast<SMFSource> (SourceFactory::createWritable (DataType::MIDI, *this, path, sample_rate())); } else { throw failed_constructor (); } @@ -4966,9 +4963,7 @@ Session::create_midi_source_by_stealing_name (boost::shared_ptr<Track> track) const string path = Glib::build_filename (source_search_path (DataType::MIDI).front(), name); - return boost::dynamic_pointer_cast<SMFSource> ( - SourceFactory::createWritable ( - DataType::MIDI, *this, path, false, sample_rate())); + return boost::dynamic_pointer_cast<SMFSource> (SourceFactory::createWritable (DataType::MIDI, *this, path, sample_rate())); } bool @@ -5726,7 +5721,7 @@ Session::write_one_track (Track& track, samplepos_t start, samplepos_t end, } try { - source = SourceFactory::createWritable (data_type, *this, path, false, sample_rate()); + source = SourceFactory::createWritable (data_type, *this, path, sample_rate()); } catch (failed_constructor& err) { diff --git a/libs/ardour/test/audio_region_test.cc b/libs/ardour/test/audio_region_test.cc index 832918054b..2eee17789f 100644 --- a/libs/ardour/test/audio_region_test.cc +++ b/libs/ardour/test/audio_region_test.cc @@ -42,7 +42,7 @@ AudioRegionTest::setUp () std::string const test_wav_path = Glib::build_filename (new_test_output_dir(), "test.wav"); _playlist = PlaylistFactory::create (DataType::AUDIO, *_session, "test"); _audio_playlist = boost::dynamic_pointer_cast<AudioPlaylist> (_playlist); - _source = SourceFactory::createWritable (DataType::AUDIO, *_session, test_wav_path, false, get_test_sample_rate ()); + _source = SourceFactory::createWritable (DataType::AUDIO, *_session, test_wav_path, get_test_sample_rate ()); /* Write a staircase to the source */ |