X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_timefx.cc;h=c623a3fe1336737bd354bb02711cba2099ed4821;hb=57d7b5de65b399fc7d1d61a3a9212c0fa7f1e95c;hp=830d8999a445b4cb67b496b7176f88385bcc054d;hpb=ddd162bdb050558f93222800ebd57d737f2c9e0d;p=ardour.git diff --git a/gtk2_ardour/editor_timefx.cc b/gtk2_ardour/editor_timefx.cc index 830d8999a4..c623a3fe13 100644 --- a/gtk2_ardour/editor_timefx.cc +++ b/gtk2_ardour/editor_timefx.cc @@ -21,14 +21,21 @@ #include #include #include - #include +#include #include "pbd/error.h" #include "pbd/pthread_utils.h" #include "pbd/memento_command.h" #include "pbd/stateful_diff_command.h" +#include "ardour/audioregion.h" +#include "ardour/midi_stretch.h" +#include "ardour/pitch.h" +#include "ardour/region.h" +#include "ardour/session.h" +#include "ardour/stretch.h" + #include #include "audio_region_view.h" @@ -37,21 +44,12 @@ #include "region_selection.h" #include "time_fx_dialog.h" -#include "ardour/session.h" -#include "ardour/region.h" -#include "ardour/audioplaylist.h" -#include "ardour/audio_track.h" -#include "ardour/audioregion.h" -#include "ardour/stretch.h" -#include "ardour/midi_stretch.h" -#include "ardour/pitch.h" - #ifdef USE_RUBBERBAND -#include "rubberband/RubberBandStretcher.h" +#include using namespace RubberBand; #endif -#include "i18n.h" +#include "pbd/i18n.h" using namespace std; using namespace ARDOUR; @@ -63,49 +61,102 @@ using namespace Gtkmm2ext; int Editor::time_stretch (RegionSelection& regions, float fraction) { - // FIXME: kludge, implement stretching of selection of both types + RegionList audio; + RegionList midi; + int aret; - if (regions.front()->region()->data_type() == DataType::AUDIO) { - // Audio, pop up timefx dialog - return time_fx (regions, fraction, false); - } else { - // MIDI, just stretch - RouteTimeAxisView* rtv = dynamic_cast (®ions.front()->get_time_axis_view()); - if (!rtv) - return -1; - - boost::shared_ptr playlist = rtv->track()->playlist(); - - ARDOUR::TimeFXRequest request; - request.time_fraction = fraction; - MidiStretch stretch(*_session, request); - begin_reversible_command ("midi stretch"); - stretch.run(regions.front()->region()); - playlist->clear_changes (); - playlist->replace_region (regions.front()->region(), stretch.results[0], - regions.front()->region()->position()); - _session->add_command (new StatefulDiffCommand (playlist)); + begin_reversible_command (_("stretch/shrink")); + + for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ++i) { + if ((*i)->region()->data_type() == DataType::AUDIO) { + audio.push_back ((*i)->region()); + } else if ((*i)->region()->data_type() == DataType::MIDI) { + midi.push_back ((*i)->region()); + } + } + + if ((aret = time_fx (audio, fraction, false)) != 0) { commit_reversible_command (); + return aret; } + set > midi_playlists_affected; + + for (RegionList::iterator i = midi.begin(); i != midi.end(); ++i) { + boost::shared_ptr playlist = (*i)->playlist(); + + if (playlist) { + playlist->clear_changes (); + } + + } + + ARDOUR::TimeFXRequest request; + request.time_fraction = fraction; + + for (RegionList::iterator i = midi.begin(); i != midi.end(); ++i) { + boost::shared_ptr playlist = (*i)->playlist(); + + if (!playlist) { + continue; + } + + MidiStretch stretch (*_session, request); + stretch.run (*i); + + playlist->replace_region (regions.front()->region(), stretch.results[0], + regions.front()->region()->position()); + midi_playlists_affected.insert (playlist); + } + + for (set >::iterator p = midi_playlists_affected.begin(); p != midi_playlists_affected.end(); ++p) { + _session->add_command (new StatefulDiffCommand (*p)); + } + + commit_reversible_command (); + return 0; } int Editor::pitch_shift (RegionSelection& regions, float fraction) { - return time_fx (regions, fraction, true); + RegionList rl; + + for (RegionSelection::iterator i = regions.begin(); i != regions.end(); ++i) { + rl.push_back ((*i)->region()); + } + + begin_reversible_command (_("pitch shift")); + + int ret = time_fx (rl, fraction, true); + + if (ret == 0) { + commit_reversible_command (); + } else { + abort_reversible_command (); + } + + return ret; } /** @param val Percentage to time stretch by; ignored if pitch-shifting. * @param pitching true to pitch shift, false to time stretch. * @return -1 in case of error, 1 if operation was cancelled by the user, 0 if everything went ok */ int -Editor::time_fx (RegionSelection& regions, float val, bool pitching) +Editor::time_fx (RegionList& regions, float val, bool pitching) { - delete current_timefx; + if (regions.empty()) { + return 0; + } + + const samplecnt_t oldlen = (samplecnt_t) (regions.front()->length()); + const samplecnt_t newlen = (samplecnt_t) (regions.front()->length() * val); + const samplecnt_t pos = regions.front()->position (); - current_timefx = new TimeFXDialog (*this, pitching); + delete current_timefx; + current_timefx = new TimeFXDialog (*this, pitching, oldlen, newlen, pos); + current_timefx->regions = regions; switch (current_timefx->run ()) { case RESPONSE_ACCEPT: @@ -116,35 +167,14 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching) } current_timefx->status = 0; - current_timefx->regions = regions; - - if (pitching) { - - float cents = current_timefx->pitch_octave_adjustment.get_value() * 1200.0; - float pitch_fraction; - cents += current_timefx->pitch_semitone_adjustment.get_value() * 100.0; - cents += current_timefx->pitch_cent_adjustment.get_value(); - - if (cents == 0.0) { - // user didn't change anything - current_timefx->hide (); - return 0; - } - - // one octave == 1200 cents - // adding one octave doubles the frequency - // ratio is 2^^octaves - - pitch_fraction = pow(2, cents/1200); - - current_timefx->request.time_fraction = 1.0; - current_timefx->request.pitch_fraction = pitch_fraction; - - } else { - - current_timefx->request.time_fraction = val; - current_timefx->request.pitch_fraction = 1.0; + current_timefx->request.time_fraction = current_timefx->get_time_fraction (); + current_timefx->request.pitch_fraction = current_timefx->get_pitch_fraction (); + if (current_timefx->request.time_fraction == 1.0 && + current_timefx->request.pitch_fraction == 1.0) { + /* nothing to do */ + current_timefx->hide (); + return 0; } #ifdef USE_RUBBERBAND @@ -178,7 +208,31 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching) } } - switch (rb_current_opt) { + int rb_mode = rb_current_opt; + + if (pitching /*&& rb_current_opt == 6*/) { + /* The timefx dialog does not show the "stretch_opts_selector" + * when pitch-shifting. So the most recently used option from + * "Time Stretch" would be used (if any). That may even be + * "resample without preserving pitch", which would be invalid. + * + * TODO: also show stretch_opts_selector when pitching (except the option + * to not preserve pitch) and use separate rb_current_opt when pitching. + * + * Actually overhaul this the dialog and processing opts below and use rubberband's + * "Crispness" levels: + * -c 0 equivalent to --no-transients --no-lamination --window-long + * -c 1 equivalent to --detector-soft --no-lamination --window-long (for piano) + * -c 2 equivalent to --no-transients --no-lamination + * -c 3 equivalent to --no-transients + * -c 4 equivalent to --bl-transients + * -c 5 default processing options + * -c 6 equivalent to --no-lamination --window-short (may be good for drums) + */ + rb_mode = 4; + } + + switch (rb_mode) { case 0: transients = NoTransients; peaklock = false; longwin = true; shortwin = false; break; @@ -198,7 +252,7 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching) transients = NoTransients; precise = true; preserve_formants = false; - current_timefx->request.pitch_fraction = 1/val; + current_timefx->request.pitch_fraction = 1.0 / current_timefx->request.time_fraction; shortwin = true; // peaklock = false; break; @@ -245,6 +299,8 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching) current_timefx->first_delete = current_timefx->signal_delete_event().connect (sigc::mem_fun (current_timefx, &TimeFXDialog::delete_in_progress)); + current_timefx->start_updates (); + if (pthread_create_and_store ("timefx", ¤t_timefx->request.thread, timefx_thread, current_timefx)) { current_timefx->hide (); error << _("timefx cannot be started - thread creation error") << endmsg; @@ -257,73 +313,61 @@ Editor::time_fx (RegionSelection& regions, float val, bool pitching) gtk_main_iteration (); } + pthread_join (current_timefx->request.thread, 0); + current_timefx->hide (); return current_timefx->status; } void -Editor::do_timefx (TimeFXDialog& dialog) +Editor::do_timefx () { - Track* t; boost::shared_ptr playlist; boost::shared_ptr new_region; - bool in_command = false; + set > playlists_affected; - uint32_t const N = dialog.regions.size (); + uint32_t const N = current_timefx->regions.size (); - for (RegionSelection::iterator i = dialog.regions.begin(); i != dialog.regions.end(); ) { - AudioRegionView* arv = dynamic_cast(*i); + for (RegionList::iterator i = current_timefx->regions.begin(); i != current_timefx->regions.end(); ++i) { + boost::shared_ptr playlist = (*i)->playlist(); - if (!arv) { - continue; + if (playlist) { + playlist->clear_changes (); } + } - boost::shared_ptr region (arv->audio_region()); - TimeAxisView* tv = &(arv->get_time_axis_view()); - RouteTimeAxisView* rtv; - RegionSelection::iterator tmp; - - tmp = i; - ++tmp; + for (RegionList::iterator i = current_timefx->regions.begin(); i != current_timefx->regions.end(); ++i) { - if ((rtv = dynamic_cast (tv)) == 0) { - i = tmp; - continue; - } + boost::shared_ptr region = boost::dynamic_pointer_cast (*i); - if ((t = dynamic_cast (rtv->route().get())) == 0) { - i = tmp; + if (!region || (playlist = region->playlist()) == 0) { continue; } - if ((playlist = t->playlist()) == 0) { - i = tmp; - continue; - } - - if (dialog.request.cancel) { + if (current_timefx->request.cancel) { /* we were cancelled */ - dialog.status = 1; + /* XXX what to do about playlists already affected ? */ + current_timefx->status = 1; return; } Filter* fx; - if (dialog.pitching) { - fx = new Pitch (*_session, dialog.request); + if (current_timefx->pitching) { + fx = new Pitch (*_session, current_timefx->request); } else { #ifdef USE_RUBBERBAND - fx = new RBStretch (*_session, dialog.request); + fx = new RBStretch (*_session, current_timefx->request); #else - fx = new STStretch (*_session, dialog.request); + fx = new STStretch (*_session, current_timefx->request); #endif } current_timefx->descend (1.0 / N); if (fx->run (region, current_timefx)) { - dialog.status = -1; - dialog.request.done = true; + current_timefx->status = -1; + current_timefx->request.done = true; delete fx; return; } @@ -331,28 +375,20 @@ Editor::do_timefx (TimeFXDialog& dialog) if (!fx->results.empty()) { new_region = fx->results.front(); - if (!in_command) { - _session->begin_reversible_command (dialog.pitching ? _("pitch shift") : _("time stretch")); - in_command = true; - } - - playlist->clear_changes (); playlist->replace_region (region, new_region, region->position()); - _session->add_command (new StatefulDiffCommand (playlist)); + playlists_affected.insert (playlist); } current_timefx->ascend (); - - i = tmp; delete fx; } - if (in_command) { - _session->commit_reversible_command (); + for (set >::iterator p = playlists_affected.begin(); p != playlists_affected.end(); ++p) { + _session->add_command (new StatefulDiffCommand (*p)); } - dialog.status = 0; - dialog.request.done = true; + current_timefx->status = 0; + current_timefx->request.done = true; } void* @@ -364,16 +400,18 @@ Editor::timefx_thread (void *arg) pthread_setcanceltype (PTHREAD_CANCEL_ASYNCHRONOUS, 0); - tsd->editor.do_timefx (*tsd); - - /* GACK! HACK! sleep for a bit so that our request buffer for the GUI - event loop doesn't die before any changes we made are processed - by the GUI ... - */ + tsd->editor.do_timefx (); - struct timespec t = { 2, 0 }; - nanosleep (&t, 0); + /* GACK! HACK! sleep for a bit so that our request buffer for the GUI + event loop doesn't die before any changes we made are processed + by the GUI ... + */ +#ifdef PLATFORM_WINDOWS + Glib::usleep(2 * G_USEC_PER_SEC); +#else + struct timespec t = { 2, 0 }; + nanosleep (&t, 0); +#endif return 0; } -