X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_markers.cc;h=996b5878bde2a92ffc1ce8b6ddd76c0789e6df33;hb=1f6c54a2f053caa8afec70f2acffdb7f7d73f09f;hp=eb82a94f6b00c862355fa23facd0ee9679dbdfde;hpb=b9185d2c07c77eccf3679ac99f6b69f8fdd79c48;p=ardour.git diff --git a/gtk2_ardour/editor_markers.cc b/gtk2_ardour/editor_markers.cc index eb82a94f6b..996b5878bd 100644 --- a/gtk2_ardour/editor_markers.cc +++ b/gtk2_ardour/editor_markers.cc @@ -31,16 +31,17 @@ #include "canvas/item.h" #include "canvas/rectangle.h" +#include "widgets/prompter.h" + #include "editor.h" #include "marker.h" #include "selection.h" #include "editing.h" #include "gui_thread.h" #include "actions.h" -#include "prompter.h" #include "editor_drag.h" -#include "i18n.h" +#include "pbd/i18n.h" using namespace std; using namespace ARDOUR; @@ -64,24 +65,32 @@ Editor::add_new_location (Location *location) { ENSURE_GUI_THREAD (*this, &Editor::add_new_location, location); - ArdourCanvas::Group* group = add_new_location_internal (location); + ArdourCanvas::Container* group = add_new_location_internal (location); /* Do a full update of the markers in this group */ update_marker_labels (group); + + if (location->is_auto_punch()) { + update_punch_range_view (); + } + + if (location->is_auto_loop()) { + update_loop_range_view (); + } } /** Add a new location, without a time-consuming update of all marker labels; * the caller must call update_marker_labels () after calling this. * @return canvas group that the location's marker was added to. */ -ArdourCanvas::Group* +ArdourCanvas::Container* Editor::add_new_location_internal (Location* location) { LocationMarkers *lam = new LocationMarkers; uint32_t color; /* make a note here of which group this marker ends up in */ - ArdourCanvas::Group* group = 0; + ArdourCanvas::Container* group = 0; if (location->is_cd_marker()) { color = location_cd_marker_color; @@ -98,10 +107,10 @@ Editor::add_new_location_internal (Location* location) if (location->is_mark()) { if (location->is_cd_marker() && ruler_cd_marker_action->get_active()) { - lam->start = new Marker (*this, *cd_marker_group, color, location->name(), Marker::Mark, location->start()); + lam->start = new ArdourMarker (*this, *cd_marker_group, color, location->name(), ArdourMarker::Mark, location->start()); group = cd_marker_group; } else { - lam->start = new Marker (*this, *marker_group, color, location->name(), Marker::Mark, location->start()); + lam->start = new ArdourMarker (*this, *marker_group, color, location->name(), ArdourMarker::Mark, location->start()); group = marker_group; } @@ -110,41 +119,41 @@ Editor::add_new_location_internal (Location* location) } else if (location->is_auto_loop()) { // transport marker - lam->start = new Marker (*this, *transport_marker_group, color, - location->name(), Marker::LoopStart, location->start()); - lam->end = new Marker (*this, *transport_marker_group, color, - location->name(), Marker::LoopEnd, location->end()); + lam->start = new ArdourMarker (*this, *transport_marker_group, color, + location->name(), ArdourMarker::LoopStart, location->start()); + lam->end = new ArdourMarker (*this, *transport_marker_group, color, + location->name(), ArdourMarker::LoopEnd, location->end()); group = transport_marker_group; } else if (location->is_auto_punch()) { // transport marker - lam->start = new Marker (*this, *transport_marker_group, color, - location->name(), Marker::PunchIn, location->start()); - lam->end = new Marker (*this, *transport_marker_group, color, - location->name(), Marker::PunchOut, location->end()); + lam->start = new ArdourMarker (*this, *transport_marker_group, color, + location->name(), ArdourMarker::PunchIn, location->start()); + lam->end = new ArdourMarker (*this, *transport_marker_group, color, + location->name(), ArdourMarker::PunchOut, location->end()); group = transport_marker_group; } else if (location->is_session_range()) { // session range - lam->start = new Marker (*this, *marker_group, color, _("start"), Marker::SessionStart, location->start()); - lam->end = new Marker (*this, *marker_group, color, _("end"), Marker::SessionEnd, location->end()); + lam->start = new ArdourMarker (*this, *marker_group, color, _("start"), ArdourMarker::SessionStart, location->start()); + lam->end = new ArdourMarker (*this, *marker_group, color, _("end"), ArdourMarker::SessionEnd, location->end()); group = marker_group; } else { // range marker if (location->is_cd_marker() && ruler_cd_marker_action->get_active()) { - lam->start = new Marker (*this, *cd_marker_group, color, - location->name(), Marker::RangeStart, location->start()); - lam->end = new Marker (*this, *cd_marker_group, color, - location->name(), Marker::RangeEnd, location->end()); + lam->start = new ArdourMarker (*this, *cd_marker_group, color, + location->name(), ArdourMarker::RangeStart, location->start()); + lam->end = new ArdourMarker (*this, *cd_marker_group, color, + location->name(), ArdourMarker::RangeEnd, location->end()); group = cd_marker_group; } else { - lam->start = new Marker (*this, *range_marker_group, color, - location->name(), Marker::RangeStart, location->start()); - lam->end = new Marker (*this, *range_marker_group, color, - location->name(), Marker::RangeEnd, location->end()); + lam->start = new ArdourMarker (*this, *range_marker_group, color, + location->name(), ArdourMarker::RangeStart, location->start()); + lam->end = new ArdourMarker (*this, *range_marker_group, color, + location->name(), ArdourMarker::RangeEnd, location->end()); group = range_marker_group; } } @@ -155,11 +164,9 @@ Editor::add_new_location_internal (Location* location) lam->show (); } - location->start_changed.connect (*this, invalidator (*this), boost::bind (&Editor::location_changed, this, _1), gui_context()); - location->end_changed.connect (*this, invalidator (*this), boost::bind (&Editor::location_changed, this, _1), gui_context()); - location->changed.connect (*this, invalidator (*this), boost::bind (&Editor::location_changed, this, _1), gui_context()); location->name_changed.connect (*this, invalidator (*this), boost::bind (&Editor::location_changed, this, _1), gui_context()); - location->FlagsChanged.connect (*this, invalidator (*this), boost::bind (&Editor::location_flags_changed, this, _1, _2), gui_context()); + location->position_lock_style_changed.connect (*this, invalidator (*this), boost::bind (&Editor::location_changed, this, _1), gui_context()); + location->FlagsChanged.connect (*this, invalidator (*this), boost::bind (&Editor::location_flags_changed, this, location), gui_context()); pair newpair; @@ -199,7 +206,12 @@ Editor::location_changed (Location *location) return; } - lam->set_name (location->name ()); + if (location->position_lock_style() == MusicTime) { + lam->set_name ("\u266B" + location->name ()); // BEAMED EIGHTH NOTES + } else { + lam->set_name (location->name ()); + } + lam->set_position (location->start(), location->end()); if (location->is_auto_loop()) { @@ -219,15 +231,15 @@ Editor::location_changed (Location *location) * lengthened) */ void -Editor::check_marker_label (Marker* m) +Editor::check_marker_label (ArdourMarker* m) { /* Get a time-ordered list of markers from the last time anything changed */ - std::list& sorted = _sorted_marker_lists[m->get_parent()]; + std::list& sorted = _sorted_marker_lists[m->get_parent()]; - list::iterator i = find (sorted.begin(), sorted.end(), m); + list::iterator i = find (sorted.begin(), sorted.end(), m); - list::iterator prev = sorted.end (); - list::iterator next = i; + list::iterator prev = sorted.end (); + list::iterator next = i; ++next; /* Look to see if the previous marker is still behind `m' in time */ @@ -294,7 +306,7 @@ Editor::check_marker_label (Marker* m) } struct MarkerComparator { - bool operator() (Marker const * a, Marker const * b) { + bool operator() (ArdourMarker const * a, ArdourMarker const * b) { return a->position() < b->position(); } }; @@ -303,16 +315,16 @@ struct MarkerComparator { void Editor::update_marker_labels () { - for (std::map >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) { + for (std::map >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) { update_marker_labels (i->first); } } /** Look at all markers in a group and update label widths */ void -Editor::update_marker_labels (ArdourCanvas::Group* group) +Editor::update_marker_labels (ArdourCanvas::Container* group) { - list& sorted = _sorted_marker_lists[group]; + list& sorted = _sorted_marker_lists[group]; if (sorted.empty()) { return; @@ -322,11 +334,14 @@ Editor::update_marker_labels (ArdourCanvas::Group* group) sorted.sort (MarkerComparator ()); - list::iterator i = sorted.begin (); + list::iterator i = sorted.begin (); - list::iterator prev = sorted.end (); - list::iterator next = i; - ++next; + list::iterator prev = sorted.end (); + list::iterator next = i; + + if (next != sorted.end()) { + ++next; + } while (i != sorted.end()) { @@ -349,16 +364,17 @@ Editor::update_marker_labels (ArdourCanvas::Group* group) } else { (*i)->set_right_label_limit (p); } + + ++next; } prev = i; ++i; - ++next; } } void -Editor::location_flags_changed (Location *location, void*) +Editor::location_flags_changed (Location *location) { ENSURE_GUI_THREAD (*this, &Editor::location_flags_changed, location, src) @@ -458,7 +474,7 @@ Editor::find_location_markers (Location *location) const } Location * -Editor::find_location_from_marker (Marker *marker, bool& is_start) const +Editor::find_location_from_marker (ArdourMarker *marker, bool& is_start) const { LocationMarkerMap::const_iterator i; @@ -477,7 +493,7 @@ Editor::find_location_from_marker (Marker *marker, bool& is_start) const } void -Editor::refresh_location_display_internal (Locations::LocationList& locations) +Editor::refresh_location_display_internal (const Locations::LocationList& locations) { /* invalidate all */ @@ -487,7 +503,7 @@ Editor::refresh_location_display_internal (Locations::LocationList& locations) /* add new ones */ - for (Locations::LocationList::iterator i = locations.begin(); i != locations.end(); ++i) { + for (Locations::LocationList::const_iterator i = locations.begin(); i != locations.end(); ++i) { LocationMarkerMap::iterator x; @@ -523,8 +539,8 @@ Editor::refresh_location_display_internal (Locations::LocationList& locations) i = tmp; } - update_punch_range_view (false); - update_loop_range_view (false); + update_punch_range_view (); + update_loop_range_view (); } void @@ -571,11 +587,11 @@ Editor::LocationMarkers::set_name (const string& str) { /* XXX: hack: don't change names of session start/end markers */ - if (start->type() != Marker::SessionStart) { + if (start->type() != ArdourMarker::SessionStart) { start->set_name (str); } - if (end && end->type() != Marker::SessionEnd) { + if (end && end->type() != ArdourMarker::SessionEnd) { end->set_name (str); } } @@ -627,30 +643,23 @@ Editor::LocationMarkers::setup_lines () } void -Editor::mouse_add_new_marker (framepos_t where, bool is_cd, bool is_xrun) +Editor::mouse_add_new_marker (framepos_t where, bool is_cd) { - string markername, markerprefix; + string markername; int flags = (is_cd ? Location::IsCDMarker|Location::IsMark : Location::IsMark); - if (is_xrun) { - markerprefix = "xrun"; - flags = Location::IsMark; - } else { - markerprefix = "mark"; - } - if (_session) { - _session->locations()->next_available_name(markername, markerprefix); - if (!is_xrun && !choose_new_marker_name(markername)) { + _session->locations()->next_available_name(markername, _("mark")); + if (!choose_new_marker_name(markername)) { return; } - Location *location = new Location (*_session, where, where, markername, (Location::Flags) flags); - _session->begin_reversible_command (_("add marker")); + Location *location = new Location (*_session, where, where, markername, (Location::Flags) flags, get_grid_music_divisions (0)); + begin_reversible_command (_("add marker")); + XMLNode &before = _session->locations()->get_state(); _session->locations()->add (location, true); XMLNode &after = _session->locations()->get_state(); _session->add_command (new MementoCommand(*(_session->locations()), &before, &after)); - _session->commit_reversible_command (); /* find the marker we just added */ @@ -659,9 +668,43 @@ Editor::mouse_add_new_marker (framepos_t where, bool is_cd, bool is_xrun) /* make it the selected marker */ selection->set (lam->start); } + + commit_reversible_command (); } } +void +Editor::mouse_add_new_loop (framepos_t where) +{ + if (!_session) { + return; + } + + /* Make this marker 1/8th of the visible area of the session so that + it's reasonably easy to manipulate after creation. + */ + + framepos_t const end = where + current_page_samples() / 8; + + set_loop_range (where, end, _("set loop range")); +} + +void +Editor::mouse_add_new_punch (framepos_t where) +{ + if (!_session) { + return; + } + + /* Make this marker 1/8th of the visible area of the session so that + it's reasonably easy to manipulate after creation. + */ + + framepos_t const end = where + current_page_samples() / 8; + + set_punch_range (where, end, _("set punch range")); +} + void Editor::mouse_add_new_range (framepos_t where) { @@ -690,12 +733,12 @@ Editor::mouse_add_new_range (framepos_t where) void Editor::remove_marker (ArdourCanvas::Item& item, GdkEvent*) { - Marker* marker; + ArdourMarker* marker; bool is_start; - if ((marker = static_cast (item.get_data ("marker"))) == 0) { + if ((marker = static_cast (item.get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } if (entered_marker == marker) { @@ -705,19 +748,19 @@ Editor::remove_marker (ArdourCanvas::Item& item, GdkEvent*) Location* loc = find_location_from_marker (marker, is_start); if (_session && loc) { - Glib::signal_idle().connect (sigc::bind (sigc::mem_fun(*this, &Editor::really_remove_marker), loc)); + Glib::signal_idle().connect (sigc::bind (sigc::mem_fun(*this, &Editor::really_remove_marker), loc)); } } gint Editor::really_remove_marker (Location* loc) { - _session->begin_reversible_command (_("remove marker")); + begin_reversible_command (_("remove marker")); XMLNode &before = _session->locations()->get_state(); _session->locations()->remove (loc); XMLNode &after = _session->locations()->get_state(); _session->add_command (new MementoCommand(*(_session->locations()), &before, &after)); - _session->commit_reversible_command (); + commit_reversible_command (); return FALSE; } @@ -729,11 +772,11 @@ Editor::location_gone (Location *location) LocationMarkerMap::iterator i; if (location == transport_loop_location()) { - update_loop_range_view (true); + update_loop_range_view (); } if (location == transport_punch_location()) { - update_punch_range_view (true); + update_punch_range_view (); } for (i = location_markers.begin(); i != location_markers.end(); ++i) { @@ -747,6 +790,13 @@ Editor::location_gone (Location *location) LocationMarkers* m = i->second; location_markers.erase (i); delete m; + + /* Markers that visually overlap with this (removed) marker + * need to be re-displayed. + * But finding such cases is similarly expensive as simply + * re-displaying all.. so: + */ + refresh_location_display (); break; } } @@ -764,25 +814,30 @@ Editor::tempo_or_meter_marker_context_menu (GdkEventButton* ev, ArdourCanvas::It bool can_remove = false; if (mm) { - can_remove = mm->meter().movable (); + can_remove = !mm->meter().initial (); + delete meter_marker_menu; + build_meter_marker_menu (mm, can_remove); + meter_marker_menu->popup (1, ev->time); } else if (tm) { - can_remove = tm->tempo().movable (); + if (!tm->tempo().active()) { + return; + } + can_remove = !tm->tempo().initial() && !tm->tempo().locked_to_meter(); + delete tempo_marker_menu; + build_tempo_marker_menu (tm, can_remove); + tempo_marker_menu->popup (1, ev->time); } else { return; } - - delete tempo_or_meter_marker_menu; - build_tempo_or_meter_marker_menu (can_remove); - tempo_or_meter_marker_menu->popup (1, ev->time); } void Editor::marker_context_menu (GdkEventButton* ev, ArdourCanvas::Item* item) { - Marker * marker; - if ((marker = reinterpret_cast (item->get_data("marker"))) == 0) { + ArdourMarker * marker; + if ((marker = reinterpret_cast (item->get_data("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } bool is_start; @@ -790,9 +845,8 @@ Editor::marker_context_menu (GdkEventButton* ev, ArdourCanvas::Item* item) if (loc == transport_loop_location() || loc == transport_punch_location() || loc->is_session_range ()) { - if (transport_marker_menu == 0) { - build_range_marker_menu (loc == transport_loop_location() || loc == transport_punch_location(), loc->is_session_range()); - } + delete transport_marker_menu; + build_range_marker_menu (loc, loc == transport_loop_location() || loc == transport_punch_location(), loc->is_session_range()); marker_menu_item = item; transport_marker_menu->popup (1, ev->time); @@ -820,9 +874,9 @@ Editor::marker_context_menu (GdkEventButton* ev, ArdourCanvas::Item* item) marker_menu->popup (1, ev->time); } else if (loc->is_range_marker()) { - if (range_marker_menu == 0) { - build_range_marker_menu (false, false); - } + delete range_marker_menu; + build_range_marker_menu (loc, false, false); + marker_menu_item = item; range_marker_menu->popup (1, ev->time); } @@ -845,6 +899,7 @@ Editor::build_marker_menu (Location* loc) using namespace Menu_Helpers; marker_menu = new Menu; + MenuList& items = marker_menu->items(); marker_menu->set_name ("ArdourContextMenu"); @@ -868,9 +923,8 @@ Editor::build_marker_menu (Location* loc) items.push_back (CheckMenuElem (_("Glue to Bars and Beats"))); Gtk::CheckMenuItem* glue_item = static_cast (&items.back()); - if (loc->position_lock_style() == MusicTime) { - glue_item->set_active (); - } + glue_item->set_active (loc->position_lock_style() == MusicTime); + glue_item->signal_activate().connect (sigc::mem_fun (*this, &Editor::toggle_marker_menu_glue)); items.push_back (SeparatorElem()); @@ -879,13 +933,14 @@ Editor::build_marker_menu (Location* loc) } void -Editor::build_range_marker_menu (bool loop_or_punch, bool session) +Editor::build_range_marker_menu (Location* loc, bool loop_or_punch, bool session) { using namespace Menu_Helpers; - bool const loop_or_punch_or_session = loop_or_punch | session; + bool const loop_or_punch_or_session = loop_or_punch || session; + + Menu* markerMenu = new Menu; - Menu *markerMenu = new Menu; if (loop_or_punch_or_session) { transport_marker_menu = markerMenu; } else { @@ -900,12 +955,17 @@ Editor::build_range_marker_menu (bool loop_or_punch, bool session) items.push_back (MenuElem (_("Loop Range"), sigc::mem_fun(*this, &Editor::marker_menu_loop_range))); items.push_back (MenuElem (_("Set Marker from Playhead"), sigc::mem_fun(*this, &Editor::marker_menu_set_from_playhead))); - if (!Profile->get_sae()) { - items.push_back (MenuElem (_("Set Range from Selection"), sigc::bind (sigc::mem_fun(*this, &Editor::marker_menu_set_from_selection), false))); - } + items.push_back (MenuElem (_("Set Range from Selection"), sigc::bind (sigc::mem_fun(*this, &Editor::marker_menu_set_from_selection), false))); items.push_back (MenuElem (_("Zoom to Range"), sigc::mem_fun (*this, &Editor::marker_menu_zoom_to_range))); + items.push_back (SeparatorElem()); + items.push_back (CheckMenuElem (_("Glue to Bars and Beats"))); + + Gtk::CheckMenuItem* glue_item = static_cast (&items.back()); + glue_item->set_active (loc->position_lock_style() == MusicTime); + glue_item->signal_activate().connect (sigc::mem_fun (*this, &Editor::toggle_marker_menu_glue)); + items.push_back (SeparatorElem()); items.push_back (MenuElem (_("Export Range..."), sigc::mem_fun(*this, &Editor::export_range))); items.push_back (SeparatorElem()); @@ -922,22 +982,69 @@ Editor::build_range_marker_menu (bool loop_or_punch, bool session) if (!loop_or_punch_or_session || !session) { items.push_back (SeparatorElem()); } - + items.push_back (MenuElem (_("Separate Regions in Range"), sigc::mem_fun(*this, &Editor::marker_menu_separate_regions_using_location))); items.push_back (MenuElem (_("Select All in Range"), sigc::mem_fun(*this, &Editor::marker_menu_select_all_selectables_using_range))); - if (!Profile->get_sae()) { - items.push_back (MenuElem (_("Select Range"), sigc::mem_fun(*this, &Editor::marker_menu_select_using_range))); + items.push_back (MenuElem (_("Select Range"), sigc::mem_fun(*this, &Editor::marker_menu_select_using_range))); +} + +void +Editor::build_tempo_marker_menu (TempoMarker* loc, bool can_remove) +{ + using namespace Menu_Helpers; + + tempo_marker_menu = new Menu; + + MenuList& items = tempo_marker_menu->items(); + tempo_marker_menu->set_name ("ArdourContextMenu"); + + if (!loc->tempo().initial()) { + if (loc->tempo().clamped()) { + items.push_back (MenuElem (_("Don't Continue"), sigc::mem_fun(*this, &Editor::toggle_tempo_clamped))); + } else { + items.push_back (MenuElem (_("Continue"), sigc::mem_fun(*this, &Editor::toggle_tempo_clamped))); + } } + + if (loc->tempo().type() == TempoSection::Ramp) { + items.push_back (MenuElem (_("Set Constant"), sigc::mem_fun(*this, &Editor::toggle_tempo_type))); + } + + TempoSection* next_ts = _session->tempo_map().next_tempo_section (&loc->tempo()); + if (next_ts && next_ts->note_types_per_minute() != loc->tempo().end_note_types_per_minute()) { + items.push_back (MenuElem (_("Ramp to Next"), sigc::mem_fun(*this, &Editor::ramp_to_next_tempo))); + } + + if (loc->tempo().position_lock_style() == AudioTime && can_remove) { + items.push_back (SeparatorElem()); + items.push_back (MenuElem (_("Lock to Music"), sigc::mem_fun(*this, &Editor::toggle_marker_lock_style))); + } else if (can_remove) { + items.push_back (SeparatorElem()); + items.push_back (MenuElem (_("Lock to Audio"), sigc::mem_fun(*this, &Editor::toggle_marker_lock_style))); + } + + items.push_back (SeparatorElem()); + + items.push_back (MenuElem (_("Edit..."), sigc::mem_fun(*this, &Editor::marker_menu_edit))); + items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &Editor::marker_menu_remove))); + items.back().set_sensitive (can_remove); } void -Editor::build_tempo_or_meter_marker_menu (bool can_remove) +Editor::build_meter_marker_menu (MeterMarker* loc, bool can_remove) { using namespace Menu_Helpers; - tempo_or_meter_marker_menu = new Menu; - MenuList& items = tempo_or_meter_marker_menu->items(); - tempo_or_meter_marker_menu->set_name ("ArdourContextMenu"); + meter_marker_menu = new Menu; + + MenuList& items = meter_marker_menu->items(); + meter_marker_menu->set_name ("ArdourContextMenu"); + + if (loc->meter().position_lock_style() == AudioTime && can_remove) { + items.push_back (MenuElem (_("Lock to Music"), sigc::mem_fun(*this, &Editor::toggle_marker_lock_style))); + } else if (can_remove) { + items.push_back (MenuElem (_("Lock to Audio"), sigc::mem_fun(*this, &Editor::toggle_marker_lock_style))); + } items.push_back (MenuElem (_("Edit..."), sigc::mem_fun(*this, &Editor::marker_menu_edit))); items.push_back (MenuElem (_("Remove"), sigc::mem_fun(*this, &Editor::marker_menu_remove))); @@ -951,6 +1058,7 @@ Editor::build_new_transport_marker_menu () using namespace Menu_Helpers; new_transport_marker_menu = new Menu; + MenuList& items = new_transport_marker_menu->items(); new_transport_marker_menu->set_name ("ArdourContextMenu"); @@ -963,11 +1071,11 @@ Editor::build_new_transport_marker_menu () void Editor::marker_menu_hide () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -981,36 +1089,36 @@ Editor::marker_menu_hide () void Editor::marker_menu_select_using_range () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; bool is_start; if (((l = find_location_from_marker (marker, is_start)) != 0) && (l->end() > l->start())) { - set_selection_from_range (*l); + set_selection_from_range (*l); } } void Editor::marker_menu_select_all_selectables_using_range () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; bool is_start; if (((l = find_location_from_marker (marker, is_start)) != 0) && (l->end() > l->start())) { - select_all_within (l->start(), l->end() - 1, 0, DBL_MAX, track_views, Selection::Set, false); + select_all_within (l->start(), l->end() - 1, 0, DBL_MAX, track_views, Selection::Set, false); } } @@ -1018,18 +1126,18 @@ Editor::marker_menu_select_all_selectables_using_range () void Editor::marker_menu_separate_regions_using_location () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; bool is_start; if (((l = find_location_from_marker (marker, is_start)) != 0) && (l->end() > l->start())) { - separate_regions_using_location (*l); + separate_regions_using_location (*l); } } @@ -1037,11 +1145,11 @@ Editor::marker_menu_separate_regions_using_location () void Editor::marker_menu_play_from () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -1067,11 +1175,11 @@ Editor::marker_menu_play_from () void Editor::marker_menu_set_playhead () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -1095,14 +1203,14 @@ Editor::marker_menu_set_playhead () void Editor::marker_menu_range_to_next () { - Marker* marker; + ArdourMarker* marker; if (!_session) { return; } - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -1128,26 +1236,27 @@ Editor::marker_menu_range_to_next () void Editor::marker_menu_set_from_playhead () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; bool is_start; + const int32_t divisions = get_grid_music_divisions (0); if ((l = find_location_from_marker (marker, is_start)) != 0) { if (l->is_mark()) { - l->set_start (_session->audible_frame ()); + l->set_start (_session->audible_frame (), false, true, divisions); } else { if (is_start) { - l->set_start (_session->audible_frame ()); + l->set_start (_session->audible_frame (), false, true, divisions); } else { - l->set_end (_session->audible_frame ()); + l->set_end (_session->audible_frame (), false, true, divisions); } } } @@ -1156,11 +1265,11 @@ Editor::marker_menu_set_from_playhead () void Editor::marker_menu_set_from_selection (bool /*force_regions*/) { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -1173,7 +1282,7 @@ Editor::marker_menu_set_from_selection (bool /*force_regions*/) // nothing for now } else { - + if (!selection->time.empty()) { l->set (selection->time.start(), selection->time.end_frame()); } else if (!selection->regions.empty()) { @@ -1187,11 +1296,11 @@ Editor::marker_menu_set_from_selection (bool /*force_regions*/) void Editor::marker_menu_play_range () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; @@ -1212,25 +1321,22 @@ Editor::marker_menu_play_range () void Editor::marker_menu_loop_range () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* l; bool is_start; if ((l = find_location_from_marker (marker, is_start)) != 0) { - Location* l2; - if ((l2 = transport_loop_location()) != 0) { - l2->set (l->start(), l->end()); - - // enable looping, reposition and start rolling - _session->request_play_loop(true); - _session->request_locate (l2->start(), true); + if (l != transport_loop_location()) { + set_loop_range (l->start(), l->end(), _("loop range from marker")); } + _session->request_locate (l->start(), true); + _session->request_play_loop (true); } } @@ -1238,7 +1344,7 @@ Editor::marker_menu_loop_range () void Editor::marker_menu_zoom_to_range () { - Marker* marker = reinterpret_cast (marker_menu_item->get_data ("marker")); + ArdourMarker* marker = reinterpret_cast (marker_menu_item->get_data ("marker")); assert (marker); bool is_start; @@ -1252,7 +1358,7 @@ Editor::marker_menu_zoom_to_range () if (a >= extra) { a -= extra; } - + framepos_t b = l->end (); if (b < (max_framepos - extra)) { b += extra; @@ -1264,10 +1370,10 @@ Editor::marker_menu_zoom_to_range () void Editor::dynamic_cast_marker_object (void* p, MeterMarker** m, TempoMarker** t) const { - Marker* marker = reinterpret_cast (p); + ArdourMarker* marker = reinterpret_cast (p); if (!marker) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } *m = dynamic_cast (marker); @@ -1304,14 +1410,137 @@ Editor::marker_menu_remove () } } +void +Editor::toggle_marker_lock_style () +{ + MeterMarker* mm; + TempoMarker* tm; + dynamic_cast_marker_object (marker_menu_item->get_data ("marker"), &mm, &tm); + + if (mm) { + begin_reversible_command (_("change meter lock style")); + XMLNode &before = _session->tempo_map().get_state(); + MeterSection* msp = &mm->meter(); + + const Meter meter (msp->divisions_per_bar(), msp->note_divisor()); + const Timecode::BBT_Time bbt (msp->bbt()); + const PositionLockStyle pls = (msp->position_lock_style() == AudioTime) ? MusicTime : AudioTime; + + _session->tempo_map().replace_meter (*msp, meter, bbt, msp->frame(), pls); + + XMLNode &after = _session->tempo_map().get_state(); + _session->add_command(new MementoCommand(_session->tempo_map(), &before, &after)); + commit_reversible_command (); + } else if (tm) { + TempoSection* tsp = &tm->tempo(); + + const double pulse = tsp->pulse(); + const framepos_t frame = tsp->frame(); + const PositionLockStyle pls = (tsp->position_lock_style() == AudioTime) ? MusicTime : AudioTime; + const Tempo tempo (tsp->note_types_per_minute(), tsp->note_type(), tsp->end_note_types_per_minute()); + + begin_reversible_command (_("change tempo lock style")); + XMLNode &before = _session->tempo_map().get_state(); + + _session->tempo_map().replace_tempo (*tsp, tempo, pulse, frame, pls); + + XMLNode &after = _session->tempo_map().get_state(); + _session->add_command(new MementoCommand(_session->tempo_map(), &before, &after)); + commit_reversible_command (); + } +} +/* actally just resets the ts to constant using initial tempo */ +void +Editor::toggle_tempo_type () +{ + TempoMarker* tm; + MeterMarker* mm; + dynamic_cast_marker_object (marker_menu_item->get_data ("marker"), &mm, &tm); + + if (tm) { + TempoSection* tsp = &tm->tempo(); + + const Tempo tempo (tsp->note_types_per_minute(), tsp->note_type()); + const double pulse = tsp->pulse(); + const framepos_t frame = tsp->frame(); + const PositionLockStyle pls = tsp->position_lock_style(); + + begin_reversible_command (_("set tempo to constant")); + XMLNode &before = _session->tempo_map().get_state(); + + _session->tempo_map().replace_tempo (*tsp, tempo, pulse, frame, pls); + + XMLNode &after = _session->tempo_map().get_state(); + _session->add_command(new MementoCommand(_session->tempo_map(), &before, &after)); + commit_reversible_command (); + } +} +/* clamped locks the previous section end tempo to the start tempo */ +void +Editor::toggle_tempo_clamped () +{ + TempoMarker* tm; + MeterMarker* mm; + dynamic_cast_marker_object (marker_menu_item->get_data ("marker"), &mm, &tm); + + if (tm) { + begin_reversible_command (_("Clamp Tempo")); + XMLNode &before = _session->tempo_map().get_state(); + + TempoSection* tsp = &tm->tempo(); + TempoSection* prev = _session->tempo_map().previous_tempo_section (tsp); + + if (prev) { + /* set to the end tempo of the previous section */ + Tempo new_tempo (prev->end_note_types_per_minute(), prev->note_type(), tsp->end_note_types_per_minute()); + _session->tempo_map().gui_change_tempo (tsp, new_tempo); + } + + tsp->set_clamped (!tsp->clamped()); + + XMLNode &after = _session->tempo_map().get_state(); + _session->add_command(new MementoCommand(_session->tempo_map(), &before, &after)); + commit_reversible_command (); + } +} + +void +Editor::ramp_to_next_tempo () +{ + TempoMarker* tm; + MeterMarker* mm; + dynamic_cast_marker_object (marker_menu_item->get_data ("marker"), &mm, &tm); + + if (tm) { + TempoMap& tmap (_session->tempo_map()); + TempoSection* tsp = &tm->tempo(); + TempoSection* next_ts = tmap.next_tempo_section (&tm->tempo()); + if (next_ts) { + const Tempo tempo (tsp->note_types_per_minute(), tsp->note_type(), next_ts->note_types_per_minute()); + const double pulse = tsp->pulse(); + const framepos_t frame = tsp->frame(); + const PositionLockStyle pls = tsp->position_lock_style(); + + begin_reversible_command (_("ramp to next tempo")); + XMLNode &before = _session->tempo_map().get_state(); + + tmap.replace_tempo (*tsp, tempo, pulse, frame, pls); + + XMLNode &after = _session->tempo_map().get_state(); + _session->add_command(new MementoCommand(_session->tempo_map(), &before, &after)); + commit_reversible_command (); + } + } +} + void Editor::toggle_marker_menu_lock () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* loc; @@ -1333,11 +1562,11 @@ Editor::toggle_marker_menu_lock () void Editor::marker_menu_rename () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } @@ -1345,20 +1574,22 @@ Editor::marker_menu_rename () } void -Editor::rename_marker(Marker *marker) +Editor::rename_marker(ArdourMarker *marker) { Location* loc; bool is_start; loc = find_location_from_marker (marker, is_start); - if (!loc) - return; + if (!loc) { + return; + } - if (loc == transport_loop_location() || loc == transport_punch_location() || loc->is_session_range()) + if (loc == transport_loop_location() || loc == transport_punch_location() || loc->is_session_range()) { return; + } - ArdourPrompter dialog (true); + ArdourWidgets::Prompter dialog (true); string txt; dialog.set_prompt (_("New Name:")); @@ -1420,7 +1651,7 @@ Editor::new_transport_marker_menu_set_punch () } void -Editor::update_loop_range_view (bool visibility) +Editor::update_loop_range_view () { if (_session == 0) { return; @@ -1436,17 +1667,15 @@ Editor::update_loop_range_view (bool visibility) transport_loop_range_rect->set_x0 (x1); transport_loop_range_rect->set_x1 (x2); - if (visibility) { - transport_loop_range_rect->show(); - } + transport_loop_range_rect->show(); - } else if (visibility) { + } else { transport_loop_range_rect->hide(); } } void -Editor::update_punch_range_view (bool visibility) +Editor::update_punch_range_view () { if (_session == 0) { return; @@ -1455,20 +1684,28 @@ Editor::update_punch_range_view (bool visibility) Location* tpl; if ((_session->config.get_punch_in() || _session->config.get_punch_out()) && ((tpl = transport_punch_location()) != 0)) { - ArdourCanvas::Rect const v = _track_canvas->visible_area (); + + double pixel_start; + double pixel_end; + if (_session->config.get_punch_in()) { - transport_punch_range_rect->set_x0 (sample_to_pixel (tpl->start())); - transport_punch_range_rect->set_x1 (_session->config.get_punch_out() ? sample_to_pixel (tpl->end()) : sample_to_pixel (JACK_MAX_FRAMES)); + pixel_start = sample_to_pixel (tpl->start()); } else { - transport_punch_range_rect->set_x0 (0); - transport_punch_range_rect->set_x1 (_session->config.get_punch_out() ? sample_to_pixel (tpl->end()) : v.width ()); + pixel_start = 0; } - - if (visibility) { - transport_punch_range_rect->show(); + if (_session->config.get_punch_out()) { + pixel_end = sample_to_pixel (tpl->end()); + } else { + pixel_end = sample_to_pixel (max_framepos); } - } else if (visibility) { - transport_punch_range_rect->hide(); + + transport_punch_range_rect->set_x0 (pixel_start); + transport_punch_range_rect->set_x1 (pixel_end); + transport_punch_range_rect->show(); + + } else { + + transport_punch_range_rect->hide(); } } @@ -1489,9 +1726,9 @@ Editor::marker_selection_changed () } struct SortLocationsByPosition { - bool operator() (Location* a, Location* b) { - return a->start() < b->start(); - } + bool operator() (Location* a, Location* b) { + return a->start() < b->start(); + } }; void @@ -1521,11 +1758,11 @@ Editor::goto_nth_marker (int n) void Editor::toggle_marker_menu_glue () { - Marker* marker; + ArdourMarker* marker; - if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { + if ((marker = reinterpret_cast (marker_menu_item->get_data ("marker"))) == 0) { fatal << _("programming error: marker canvas item has no marker object pointer!") << endmsg; - /*NOTREACHED*/ + abort(); /*NOTREACHED*/ } Location* loc; @@ -1537,12 +1774,18 @@ Editor::toggle_marker_menu_glue () return; } + begin_reversible_command (_("change marker lock style")); + XMLNode &before = _session->locations()->get_state(); + if (loc->position_lock_style() == MusicTime) { loc->set_position_lock_style (AudioTime); } else { loc->set_position_lock_style (MusicTime); } + XMLNode &after = _session->locations()->get_state(); + _session->add_command(new MementoCommand(*(_session->locations()), &before, &after)); + commit_reversible_command (); } void @@ -1556,14 +1799,14 @@ Editor::toggle_marker_lines () } void -Editor::remove_sorted_marker (Marker* m) +Editor::remove_sorted_marker (ArdourMarker* m) { - for (std::map >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) { + for (std::map >::iterator i = _sorted_marker_lists.begin(); i != _sorted_marker_lists.end(); ++i) { i->second.remove (m); } } -Marker * +ArdourMarker * Editor::find_marker_from_location_id (PBD::ID const & id, bool is_start) const { for (LocationMarkerMap::const_iterator i = location_markers.begin(); i != location_markers.end(); ++i) {