summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
-rw-r--r--gtk2_ardour/editor.cc86
-rw-r--r--gtk2_ardour/editor.h9
-rw-r--r--gtk2_ardour/editor_actions.cc2
-rw-r--r--gtk2_ardour/editor_ops.cc52
-rw-r--r--gtk2_ardour/midi_region_view.cc6
-rw-r--r--gtk2_ardour/midi_region_view.h5
-rw-r--r--gtk2_ardour/public_editor.h3
-rw-r--r--gtk2_ardour/region_view.h2
-rw-r--r--libs/ardour/SConscript3
-rw-r--r--libs/ardour/ardour/audioregion.h6
-rw-r--r--libs/ardour/ardour/filter.h (renamed from libs/ardour/ardour/audiofilter.h)31
-rw-r--r--libs/ardour/ardour/region.h3
-rw-r--r--libs/ardour/ardour/reverse.h6
-rw-r--r--libs/ardour/ardour/session.h2
-rw-r--r--libs/ardour/audioregion.cc8
-rw-r--r--libs/ardour/filter.cc (renamed from libs/ardour/audiofilter.cc)35
-rw-r--r--libs/ardour/region.cc8
-rw-r--r--libs/ardour/reverse.cc8
-rw-r--r--libs/ardour/session_state.cc15
-rw-r--r--libs/ardour/session_timefx.cc2
20 files changed, 195 insertions, 97 deletions
diff --git a/gtk2_ardour/editor.cc b/gtk2_ardour/editor.cc
index bb7ae142ec..8b087e7096 100644
--- a/gtk2_ardour/editor.cc
+++ b/gtk2_ardour/editor.cc
@@ -50,7 +50,7 @@
#include <ardour/location.h>
#include <ardour/audioplaylist.h>
#include <ardour/audioregion.h>
-#include <ardour/region.h>
+#include <ardour/midi_region.h>
#include <ardour/session_route.h>
#include <ardour/tempo.h>
#include <ardour/utils.h>
@@ -1531,7 +1531,7 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items)
while (i != selection->regions.end() && boost::dynamic_pointer_cast<AudioRegion>((*i)->region()) == 0) {
++i;
}
- bool const have_selected_audio_region = (i != selection->regions.end());
+ const bool have_selected_audio_region = (i != selection->regions.end());
if (have_selected_audio_region) {
@@ -1551,9 +1551,19 @@ Editor::add_region_context_items (Menu_Helpers::MenuList& edit_items)
items.push_back (MenuElem (_("Normalize"), mem_fun (*this, &Editor::normalize_regions)));
}
- items.push_back (MenuElem (_("Reverse"), mem_fun(*this, &Editor::reverse_regions)));
- items.push_back (SeparatorElem());
+ /* Find out if we have a selected MIDI region */
+ i = selection->regions.begin();
+ while (i != selection->regions.end() && boost::dynamic_pointer_cast<MidiRegion>((*i)->region()) == 0) {
+ ++i;
+ }
+ const bool have_selected_midi_region = (i != selection->regions.end());
+
+ if (have_selected_midi_region) {
+
+ items.push_back (MenuElem (_("Quantize"), mem_fun(*this, &Editor::quantize_regions)));
+ items.push_back (SeparatorElem());
+ }
/* range related stuff */
@@ -2200,24 +2210,24 @@ Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
break;
case SnapToAThirtysecondBeat:
- start = session->tempo_map().round_to_beat_subdivision (start, 32);
- break;
+ start = session->tempo_map().round_to_beat_subdivision (start, 32);
+ break;
case SnapToASixteenthBeat:
- start = session->tempo_map().round_to_beat_subdivision (start, 16);
- break;
+ start = session->tempo_map().round_to_beat_subdivision (start, 16);
+ break;
case SnapToAEighthBeat:
- start = session->tempo_map().round_to_beat_subdivision (start, 8);
- break;
+ start = session->tempo_map().round_to_beat_subdivision (start, 8);
+ break;
case SnapToAQuarterBeat:
- start = session->tempo_map().round_to_beat_subdivision (start, 4);
- break;
+ start = session->tempo_map().round_to_beat_subdivision (start, 4);
+ break;
- case SnapToAThirdBeat:
- start = session->tempo_map().round_to_beat_subdivision (start, 3);
- break;
+ case SnapToAThirdBeat:
+ start = session->tempo_map().round_to_beat_subdivision (start, 3);
+ break;
case SnapToEditCursor:
start = edit_cursor->current_frame;
@@ -2308,6 +2318,52 @@ Editor::snap_to (nframes64_t& start, int32_t direction, bool for_mark)
}
}
+double
+Editor::snap_length_beats (nframes_t start)
+{
+ if (!session) {
+ return 1.0;
+ }
+
+ const nframes64_t one_second = session->frame_rate();
+ const nframes64_t one_minute = session->frame_rate() * 60;
+ const nframes64_t one_smpte_second = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame());
+ nframes64_t one_smpte_minute = (nframes64_t)(rint(session->smpte_frames_per_second()) * session->frames_per_smpte_frame() * 60);
+ nframes64_t presnap = start;
+
+ /* FIXME: This could/should also work with non-tempo based snap settings (ie seconds) */
+
+ switch (snap_type) {
+ case SnapToBar:
+ return session->tempo_map().meter_at(start).beats_per_bar();
+
+ case SnapToBeat:
+ return 1.0;
+
+ case SnapToAThirtysecondBeat:
+ return 1.0 / (double)32.0;
+ break;
+
+ case SnapToASixteenthBeat:
+ return 1.0 / (double)16.0;
+ break;
+
+ case SnapToAEighthBeat:
+ return 1.0 / (double)8.0;
+ break;
+
+ case SnapToAQuarterBeat:
+ return 1.0 / (double)4.0;
+ break;
+
+ case SnapToAThirdBeat:
+ return 1.0 / (double)3.0;
+
+ default:
+ return 1.0;
+ }
+}
+
void
Editor::setup_toolbar ()
{
diff --git a/gtk2_ardour/editor.h b/gtk2_ardour/editor.h
index 62d25cb5e5..8bf814a472 100644
--- a/gtk2_ardour/editor.h
+++ b/gtk2_ardour/editor.h
@@ -71,7 +71,7 @@ namespace ARDOUR {
class TempoSection;
class NamedSelection;
class Session;
- class AudioFilter;
+ class Filter;
class Crossfade;
class ChanCount;
}
@@ -746,7 +746,7 @@ class Editor : public PublicEditor
/* snapshots */
- Gtk::ScrolledWindow snapshot_display_scroller;
+ Gtk::ScrolledWindow snapshot_display_scroller;
struct SnapshotDisplayModelColumns : public Gtk::TreeModel::ColumnRecord {
SnapshotDisplayModelColumns() {
add (visible_name);
@@ -892,6 +892,7 @@ class Editor : public PublicEditor
void reverse_regions ();
void normalize_regions ();
void denormalize_regions ();
+ void quantize_regions ();
void audition_region_from_region_list ();
void hide_region_from_region_list ();
@@ -1288,6 +1289,8 @@ class Editor : public PublicEditor
snap_to (first64, direction, for_mark);
first = (nframes_t) first64;
}
+
+ double snap_length_beats (nframes_t start);
uint32_t bbt_beat_subdivision;
@@ -1805,7 +1808,7 @@ class Editor : public PublicEditor
/* audio filters */
- void apply_filter (ARDOUR::AudioFilter&, string cmd);
+ void apply_filter (ARDOUR::Filter&, string cmd);
/* handling cleanup */
diff --git a/gtk2_ardour/editor_actions.cc b/gtk2_ardour/editor_actions.cc
index 86778fa11f..ee99ff6892 100644
--- a/gtk2_ardour/editor_actions.cc
+++ b/gtk2_ardour/editor_actions.cc
@@ -254,6 +254,8 @@ Editor::register_actions ()
ActionManager::session_sensitive_actions.push_back (act);
act = ActionManager::register_action (editor_actions, "normalize-region", _("Normalize Regions"), mem_fun(*this, &Editor::normalize_regions));
ActionManager::session_sensitive_actions.push_back (act);
+ act = ActionManager::register_action (editor_actions, "quantize-region", _("Quantize Regions"), mem_fun(*this, &Editor::quantize_regions));
+ ActionManager::session_sensitive_actions.push_back (act);
act = ActionManager::register_action (editor_actions, "crop", _("crop"), mem_fun(*this, &Editor::crop_region_to_selection));
ActionManager::session_sensitive_actions.push_back (act);
act = ActionManager::register_action (editor_actions, "insert-chunk", _("Insert Chunk"), bind (mem_fun(*this, &Editor::paste_named_selection), 1.0f));
diff --git a/gtk2_ardour/editor_ops.cc b/gtk2_ardour/editor_ops.cc
index ccfcdbd6c5..a82ca0260a 100644
--- a/gtk2_ardour/editor_ops.cc
+++ b/gtk2_ardour/editor_ops.cc
@@ -48,6 +48,7 @@
#include <ardour/region_factory.h>
#include <ardour/playlist_factory.h>
#include <ardour/reverse.h>
+#include <ardour/quantize.h>
#include "ardour_ui.h"
#include "editor.h"
@@ -56,6 +57,7 @@
#include "automation_time_axis.h"
#include "streamview.h"
#include "audio_region_view.h"
+#include "midi_region_view.h"
#include "rgb_macros.h"
#include "selection_templates.h"
#include "selection.h"
@@ -3281,8 +3283,21 @@ Editor::reverse_regions ()
apply_filter (rev, _("reverse regions"));
}
+
+void
+Editor::quantize_regions ()
+{
+ if (!session) {
+ return;
+ }
+
+ // FIXME: varying meter?
+ Quantize quant (*session, snap_length_beats(0));
+ apply_filter (quant, _("quantize regions"));
+}
+
void
-Editor::apply_filter (AudioFilter& filter, string command)
+Editor::apply_filter (Filter& filter, string command)
{
if (selection->regions.empty()) {
return;
@@ -3293,26 +3308,31 @@ Editor::apply_filter (AudioFilter& filter, string command)
track_canvas.get_window()->set_cursor (*wait_cursor);
gdk_flush ();
+ /* this is ugly. */
for (RegionSelection::iterator r = selection->regions.begin(); r != selection->regions.end(); ) {
- AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
- if (!arv)
- continue;
+ RegionSelection::iterator tmp = r;
+ ++tmp;
- boost::shared_ptr<Playlist> playlist = arv->region()->playlist();
+ MidiRegionView* const mrv = dynamic_cast<MidiRegionView*>(*r);
+ if (mrv) {
+ if (mrv->midi_region()->apply(filter) == 0) {
+ mrv->redisplay_model();
+ }
+ }
- RegionSelection::iterator tmp;
-
- tmp = r;
- ++tmp;
+ AudioRegionView* const arv = dynamic_cast<AudioRegionView*>(*r);
+ if (arv) {
+ boost::shared_ptr<Playlist> playlist = arv->region()->playlist();
- if (arv->audio_region()->apply (filter) == 0) {
+ if (arv->audio_region()->apply (filter) == 0) {
- XMLNode &before = playlist->get_state();
- playlist->replace_region (arv->region(), filter.results.front(), arv->region()->position());
- XMLNode &after = playlist->get_state();
- session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
- } else {
- goto out;
+ XMLNode &before = playlist->get_state();
+ playlist->replace_region (arv->region(), filter.results.front(), arv->region()->position());
+ XMLNode &after = playlist->get_state();
+ session->add_command(new MementoCommand<Playlist>(*playlist, &before, &after));
+ } else {
+ goto out;
+ }
}
r = tmp;
diff --git a/gtk2_ardour/midi_region_view.cc b/gtk2_ardour/midi_region_view.cc
index 5937b12ad9..64693c7e6f 100644
--- a/gtk2_ardour/midi_region_view.cc
+++ b/gtk2_ardour/midi_region_view.cc
@@ -423,12 +423,6 @@ MidiRegionView::set_y_position_and_height (double y, double h)
}
}
-void
-MidiRegionView::show_region_editor ()
-{
- cerr << "No MIDI region editor." << endl;
-}
-
GhostRegion*
MidiRegionView::add_ghost (AutomationTimeAxisView& atv)
{
diff --git a/gtk2_ardour/midi_region_view.h b/gtk2_ardour/midi_region_view.h
index d8c62abfe5..f74272a409 100644
--- a/gtk2_ardour/midi_region_view.h
+++ b/gtk2_ardour/midi_region_view.h
@@ -68,8 +68,8 @@ class MidiRegionView : public RegionView
{ return midi_view()->midi_view(); }
void set_y_position_and_height (double, double);
-
- void show_region_editor ();
+
+ void redisplay_model();
GhostRegion* add_ghost (AutomationTimeAxisView&);
@@ -164,7 +164,6 @@ class MidiRegionView : public RegionView
private:
- void redisplay_model();
void clear_events();
bool canvas_event(GdkEvent* ev);
diff --git a/gtk2_ardour/public_editor.h b/gtk2_ardour/public_editor.h
index 746df6aaa9..97618dcbc2 100644
--- a/gtk2_ardour/public_editor.h
+++ b/gtk2_ardour/public_editor.h
@@ -115,6 +115,9 @@ class PublicEditor : public Gtk::Window, public PBD::StatefulThingWithGoingAway
/** Snap a value according to the current snap setting. */
virtual void snap_to (nframes_t& first, int32_t direction = 0, bool for_mark = false) = 0;
+
+ /** Get the current snap value in beats */
+ virtual double snap_length_beats (nframes_t start) = 0;
/** Undo some transactions.
* @param n Number of transactions to undo.
diff --git a/gtk2_ardour/region_view.h b/gtk2_ardour/region_view.h
index 426529c528..28a2b11c0c 100644
--- a/gtk2_ardour/region_view.h
+++ b/gtk2_ardour/region_view.h
@@ -70,7 +70,7 @@ class RegionView : public TimeAxisViewItem
bool set_position(nframes_t pos, void* src, double* delta = 0);
void fake_set_opaque (bool yn);
- virtual void show_region_editor () = 0;
+ virtual void show_region_editor () {}
virtual void hide_region_editor();
virtual void region_changed (ARDOUR::Change);
diff --git a/libs/ardour/SConscript b/libs/ardour/SConscript
index 5d56e198fb..a0070117e1 100644
--- a/libs/ardour/SConscript
+++ b/libs/ardour/SConscript
@@ -52,8 +52,8 @@ buffer_set.cc
meter.cc
amp.cc
panner.cc
+filter.cc
audiofilesource.cc
-audiofilter.cc
audioregion.cc
audiosource.cc
midi_source.cc
@@ -99,6 +99,7 @@ recent_sessions.cc
region.cc
region_factory.cc
reverse.cc
+quantize.cc
route.cc
route_group.cc
send.cc
diff --git a/libs/ardour/ardour/audioregion.h b/libs/ardour/ardour/audioregion.h
index 52a07679af..ddda1f4321 100644
--- a/libs/ardour/ardour/audioregion.h
+++ b/libs/ardour/ardour/audioregion.h
@@ -38,7 +38,7 @@ namespace ARDOUR {
class Route;
class Playlist;
class Session;
-class AudioFilter;
+class Filter;
class AudioSource;
class AudioRegion : public Region
@@ -115,10 +115,6 @@ class AudioRegion : public Region
int separate_by_channel (ARDOUR::Session&, vector<boost::shared_ptr<AudioRegion> >&) const;
- /* filter */
-
- int apply (AudioFilter&);
-
/* export */
int exportme (ARDOUR::Session&, ARDOUR::AudioExportSpecification&);
diff --git a/libs/ardour/ardour/audiofilter.h b/libs/ardour/ardour/filter.h
index 6b60544942..0e4c6b3de0 100644
--- a/libs/ardour/ardour/audiofilter.h
+++ b/libs/ardour/ardour/filter.h
@@ -1,5 +1,6 @@
/*
- Copyright (C) 2004 Paul Davis
+ Copyright (C) 2007 Paul Davis
+ Author: Dave Robillard
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
@@ -17,34 +18,34 @@
*/
-#ifndef __ardour_audiofilter_h__
-#define __ardour_audiofilter_h__
+#ifndef __ardour_filter_h__
+#define __ardour_filter_h__
#include <vector>
-#include <ardour/audioregion.h>
+#include <ardour/region.h>
namespace ARDOUR {
-class AudioRegion;
+class Region;
class Session;
-class AudioFilter {
+class Filter {
public:
- AudioFilter (ARDOUR::Session& s)
- : session (s){}
- virtual ~AudioFilter() {}
+ virtual ~Filter() {}
- virtual int run (boost::shared_ptr<ARDOUR::AudioRegion>) = 0;
- std::vector<boost::shared_ptr<ARDOUR::AudioRegion> > results;
+ virtual int run (boost::shared_ptr<ARDOUR::Region>) = 0;
+ std::vector<boost::shared_ptr<ARDOUR::Region> > results;
protected:
- ARDOUR::Session& session;
+ Filter (ARDOUR::Session& s) : session(s) {}
- int make_new_sources (boost::shared_ptr<ARDOUR::AudioRegion>, ARDOUR::SourceList&);
- int finish (boost::shared_ptr<ARDOUR::AudioRegion>, ARDOUR::SourceList&);
+ int make_new_sources (boost::shared_ptr<ARDOUR::Region>, ARDOUR::SourceList&);
+ int finish (boost::shared_ptr<ARDOUR::Region>, ARDOUR::SourceList&);
+
+ ARDOUR::Session& session;
};
} /* namespace */
-#endif /* __ardour_audiofilter_h__ */
+#endif /* __ardour_filter_h__ */
diff --git a/libs/ardour/ardour/region.h b/libs/ardour/ardour/region.h
index c3aac3dfa8..e6b799fdcf 100644
--- a/libs/ardour/ardour/region.h
+++ b/libs/ardour/ardour/region.h
@@ -35,6 +35,7 @@ class XMLNode;
namespace ARDOUR {
class Playlist;
+class Filter;
enum RegionEditState {
EditChangesNothing = 0,
@@ -165,6 +166,8 @@ class Region : public PBD::StatefulDestructible, public boost::enable_shared_fro
void set_opaque (bool yn);
void set_locked (bool yn);
void set_position_locked (bool yn);
+
+ int apply (Filter&);
virtual uint32_t read_data_count() const { return _read_data_count; }
diff --git a/libs/ardour/ardour/reverse.h b/libs/ardour/ardour/reverse.h
index 3296c77c62..7870b5aa2e 100644
--- a/libs/ardour/ardour/reverse.h
+++ b/libs/ardour/ardour/reverse.h
@@ -20,16 +20,16 @@
#ifndef __ardour_reverse_h__
#define __ardour_reverse_h__
-#include <ardour/audiofilter.h>
+#include <ardour/filter.h>
namespace ARDOUR {
-class Reverse : public AudioFilter {
+class Reverse : public Filter {
public:
Reverse (ARDOUR::Session&);
~Reverse ();
- int run (boost::shared_ptr<ARDOUR::AudioRegion>);
+ int run (boost::shared_ptr<ARDOUR::Region>);
};
} /* namespace */
diff --git a/libs/ardour/ardour/session.h b/libs/ardour/ardour/session.h
index 05d87620f2..63be24d9f7 100644
--- a/libs/ardour/ardour/session.h
+++ b/libs/ardour/ardour/session.h
@@ -553,7 +553,7 @@ class Session : public PBD::StatefulDestructible
int region_name (string& result, string base = string(""), bool newlevel = false) const;
string new_region_name (string);
- string path_from_region_name (string name, string identifier);
+ string path_from_region_name (DataType type, string name, string identifier);
boost::shared_ptr<Region> find_whole_file_parent (boost::shared_ptr<Region const>);
diff --git a/libs/ardour/audioregion.cc b/libs/ardour/audioregion.cc
index dcf17ba2ea..58ece331c3 100644
--- a/libs/ardour/audioregion.cc
+++ b/libs/ardour/audioregion.cc
@@ -38,7 +38,6 @@
#include <ardour/gain.h>
#include <ardour/dB.h>
#include <ardour/playlist.h>
-#include <ardour/audiofilter.h>
#include <ardour/audiofilesource.h>
#include <ardour/region_factory.h>
#include <ardour/runtime_functions.h>
@@ -977,13 +976,6 @@ AudioRegion::separate_by_channel (Session& session, vector<boost::shared_ptr<Aud
return 0;
}
-int
-AudioRegion::apply (AudioFilter& filter)
-{
- boost::shared_ptr<AudioRegion> ar = boost::dynamic_pointer_cast<AudioRegion> (shared_from_this());
- return filter.run (ar);
-}
-
nframes_t
AudioRegion::read_raw_internal (Sample* buf, nframes_t pos, nframes_t cnt) const
{
diff --git a/libs/ardour/audiofilter.cc b/libs/ardour/filter.cc
index d4d59d4a60..f655f71668 100644
--- a/libs/ardour/audiofilter.cc
+++ b/libs/ardour/filter.cc
@@ -1,5 +1,5 @@
/*
- Copyright (C) 2004 Paul Davis
+ Copyright (C) 2004-2007 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
@@ -22,9 +22,10 @@
#include <pbd/basename.h>
#include <ardour/sndfilesource.h>
+#include <ardour/smf_source.h>
#include <ardour/session.h>
-#include <ardour/audioregion.h>
-#include <ardour/audiofilter.h>
+#include <ardour/region.h>
+#include <ardour/filter.h>
#include <ardour/region_factory.h>
#include <ardour/source_factory.h>
@@ -34,27 +35,28 @@ using namespace ARDOUR;
using namespace PBD;
int
-AudioFilter::make_new_sources (boost::shared_ptr<AudioRegion> region, SourceList& nsrcs)
+Filter::make_new_sources (boost::shared_ptr<Region> region, SourceList& nsrcs)
{
vector<string> names = region->master_source_names();
for (uint32_t i = 0; i < region->n_channels(); ++i) {
- string path = session.path_from_region_name (PBD::basename_nosuffix (names[i]), string (""));
+ string path = session.path_from_region_name (region->data_type(),
+ PBD::basename_nosuffix (names[i]), string (""));
if (path.length() == 0) {
- error << string_compose (_("audiofilter: error creating name for new audio file based on %1"), region->name())
+ error << string_compose (_("filter: error creating name for new file based on %1"), region->name())
<< endmsg;
return -1;
}
try {
- nsrcs.push_back (boost::dynamic_pointer_cast<AudioSource> (
- SourceFactory::createWritable (DataType::AUDIO, session, path, false, session.frame_rate())));
+ nsrcs.push_back (boost::dynamic_pointer_cast<Source> (
+ SourceFactory::createWritable (region->data_type(), session, path, false, session.frame_rate())));
}
catch (failed_constructor& err) {
- error << string_compose (_("audiofilter: error creating new audio file %1 (%2)"), path, strerror (errno)) << endmsg;
+ error << string_compose (_("filter: error creating new file %1 (%2)"), path, strerror (errno)) << endmsg;
return -1;
}
}
@@ -63,7 +65,7 @@ AudioFilter::make_new_sources (boost::shared_ptr<AudioRegion> region, SourceList
}
int
-AudioFilter::finish (boost::shared_ptr<AudioRegion> region, SourceList& nsrcs)
+Filter::finish (boost::shared_ptr<Region> region, SourceList& nsrcs)
{
string region_name;
@@ -75,20 +77,29 @@ AudioFilter::finish (boost::shared_ptr<AudioRegion> region, SourceList& nsrcs)
time (&xnow);
now = localtime (&xnow);
+ /* this is ugly. */
for (SourceList::iterator si = nsrcs.begin(); si != nsrcs.end(); ++si) {
boost::shared_ptr<AudioFileSource> afs = boost::dynamic_pointer_cast<AudioFileSource>(*si);
if (afs) {
afs->update_header (region->position(), *now, xnow);
afs->mark_immutable ();
}
+
+ boost::shared_ptr<SMFSource> smfs = boost::dynamic_pointer_cast<SMFSource>(*si);
+ if (smfs) {
+ smfs->update_header (region->position(), *now, xnow);
+ smfs->flush_footer ();
+ }
}
/* create a new region */
region_name = session.new_region_name (region->name());
results.clear ();
- results.push_back (boost::dynamic_pointer_cast<AudioRegion> (RegionFactory::create (nsrcs, 0, region->length(), region_name, 0,
- Region::Flag (Region::WholeFile|Region::DefaultFlags))));
+ results.push_back (RegionFactory::create (nsrcs, 0, region->length(), region_name, 0,
+ Region::Flag (Region::WholeFile|Region::DefaultFlags)));
return 0;
}
+
+
diff --git a/libs/ardour/region.cc b/libs/ardour/region.cc
index 8ac101282d..a828460bce 100644
--- a/libs/ardour/region.cc
+++ b/libs/ardour/region.cc
@@ -34,6 +34,7 @@
#include <ardour/session.h>
#include <ardour/source.h>
#include <ardour/region_factory.h>
+#include <ardour/filter.h>
#include "i18n.h"
@@ -1265,3 +1266,10 @@ Region::get_parent() const
return boost::shared_ptr<Region>();
}
+int
+Region::apply (Filter& filter)
+{
+ return filter.run (shared_from_this());
+}
+
+
diff --git a/libs/ardour/reverse.cc b/libs/ardour/reverse.cc
index c7ebecea31..02ec2924b0 100644
--- a/libs/ardour/reverse.cc
+++ b/libs/ardour/reverse.cc
@@ -33,7 +33,7 @@ using namespace std;
using namespace ARDOUR;
Reverse::Reverse (Session& s)
- : AudioFilter (s)
+ : Filter (s)
{
}
@@ -42,7 +42,7 @@ Reverse::~Reverse ()
}
int
-Reverse::run (boost::shared_ptr<AudioRegion> region)
+Reverse::run (boost::shared_ptr<Region> r)
{
SourceList nsrcs;
SourceList::iterator si;
@@ -53,6 +53,10 @@ Reverse::run (boost::shared_ptr<AudioRegion> region)
nframes_t to_read;
int ret = -1;
+ boost::shared_ptr<AudioRegion> region = boost::dynamic_pointer_cast<AudioRegion>(r);
+ if (!region)
+ return ret;
+
/* create new sources */
if (make_new_sources (region, nsrcs)) {
diff --git a/libs/ardour/session_state.cc b/libs/ardour/session_state.cc
index f6debff123..a7c8dc0200 100644
--- a/libs/ardour/session_state.cc
+++ b/libs/ardour/session_state.cc
@@ -1502,19 +1502,24 @@ Session::get_sources_as_xml ()
}
string
-Session::path_from_region_name (string name, string identifier)
+Session::path_from_region_name (DataType type, string name, string identifier)
{
char buf[PATH_MAX+1];
uint32_t n;
SessionDirectory sdir(get_best_session_directory_for_new_source());
- string sound_dir = sdir.sound_path().to_string();
+ string sound_dir = ((type == DataType::AUDIO)
+ ? sdir.sound_path().to_string()
+ : sdir.midi_path().to_string());
+
+ string ext = ((type == DataType::AUDIO) ? ".wav" : ".mid");
for (n = 0; n < 999999; ++n) {
if (identifier.length()) {
- snprintf (buf, sizeof(buf), "%s/%s%s%" PRIu32 ".wav", sound_dir.c_str(), name.c_str(),
- identifier.c_str(), n);
+ snprintf (buf, sizeof(buf), "%s/%s%s%" PRIu32 "%s", sound_dir.c_str(), name.c_str(),
+ identifier.c_str(), n, ext.c_str());
} else {
- snprintf (buf, sizeof(buf), "%s/%s-%" PRIu32 ".wav", sound_dir.c_str(), name.c_str(), n);
+ snprintf (buf, sizeof(buf), "%s/%s-%" PRIu32 "%s", sound_dir.c_str(), name.c_str(),
+ n, ext.c_str());
}
if (!Glib::file_test (buf, Glib::FILE_TEST_EXISTS)) {
diff --git a/libs/ardour/session_timefx.cc b/libs/ardour/session_timefx.cc
index aac77a59c7..115d3eeeec 100644
--- a/libs/ardour/session_timefx.cc
+++ b/libs/ardour/session_timefx.cc
@@ -85,7 +85,7 @@ Session::tempoize_region (TimeStretchRequest& tsr)
rstr = names[i];
}
- string path = path_from_region_name (PBD::basename_nosuffix (rstr), ident);
+ string path = path_from_region_name (DataType::AUDIO, PBD::basename_nosuffix (rstr), ident);
if (path.length() == 0) {
error << string_compose (_("tempoize: error creating name for new audio file based on %1"), tsr.region->name())