X-Git-Url: https://main.carlh.net/gitweb/?a=blobdiff_plain;f=gtk2_ardour%2Feditor_tempodisplay.cc;h=e0eba936c81ca160fab835b87fa622e591c22f72;hb=ccf58b8de23619db9983a45f696fd97e13a15cce;hp=1e5c02f5d8bcf2d2ca1106f1c14485a9892cb82e;hpb=68e943265edf04e63a8e8b8f62bab20f99d9c637;p=ardour.git diff --git a/gtk2_ardour/editor_tempodisplay.cc b/gtk2_ardour/editor_tempodisplay.cc index 1e5c02f5d8..e0eba936c8 100644 --- a/gtk2_ardour/editor_tempodisplay.cc +++ b/gtk2_ardour/editor_tempodisplay.cc @@ -1,5 +1,5 @@ /* - Copyright (C) 2002 Paul Davis + Copyright (C) 2002 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 @@ -17,7 +17,7 @@ */ -#include // for sprintf, grrr +#include // for sprintf, grrr #include #include #include @@ -25,14 +25,14 @@ #include -#include -#include +#include "pbd/error.h" +#include "pbd/memento_command.h" #include #include -#include -#include +#include "ardour/session.h" +#include "ardour/tempo.h" #include #include @@ -44,6 +44,7 @@ #include "gui_thread.h" #include "time_axis_view.h" #include "ardour_ui.h" +#include "tempo_lines.h" #include "i18n.h" @@ -64,7 +65,7 @@ Editor::remove_metric_marks () delete_when_idle (*x); } metric_marks.clear (); -} +} void Editor::draw_metric_marks (const Metrics& metrics) @@ -73,21 +74,21 @@ Editor::draw_metric_marks (const Metrics& metrics) const MeterSection *ms; const TempoSection *ts; char buf[64]; - + remove_metric_marks (); - + for (Metrics::const_iterator i = metrics.begin(); i != metrics.end(); ++i) { - + if ((ms = dynamic_cast(*i)) != 0) { snprintf (buf, sizeof(buf), "%g/%g", ms->beats_per_bar(), ms->note_divisor ()); - metric_marks.push_back (new MeterMarker (*this, *meter_group, ARDOUR_UI::config()->canvasvar_MeterMarker.get(), buf, + metric_marks.push_back (new MeterMarker (*this, *meter_group, ARDOUR_UI::config()->canvasvar_MeterMarker.get(), buf, *(const_cast(ms)))); } else if ((ts = dynamic_cast(*i)) != 0) { snprintf (buf, sizeof (buf), "%.2f", ts->beats_per_minute()); - metric_marks.push_back (new TempoMarker (*this, *tempo_group, ARDOUR_UI::config()->canvasvar_TempoMarker.get(), buf, + metric_marks.push_back (new TempoMarker (*this, *tempo_group, ARDOUR_UI::config()->canvasvar_TempoMarker.get(), buf, *(const_cast(ts)))); } - + } } @@ -101,12 +102,12 @@ Editor::tempo_map_changed (Change ignored) ENSURE_GUI_THREAD(bind (mem_fun (*this, &Editor::tempo_map_changed), ignored)); - compute_current_bbt_points(leftmost_frame, leftmost_frame + (nframes_t)(edit_packer.get_width() * frames_per_unit)); + if (tempo_lines) + tempo_lines->tempo_map_changed(); + + compute_current_bbt_points(leftmost_frame, leftmost_frame + current_page_frames()); session->tempo_map().apply_with_metrics (*this, &Editor::draw_metric_marks); // redraw metric markers - update_tempo_based_rulers (); - if (tempo_map_change_idle_handler_id < 0) { - tempo_map_change_idle_handler_id = Glib::signal_idle().connect (mem_fun (*this, &Editor::redraw_measures)); - } + redraw_measures (); } void @@ -115,24 +116,20 @@ Editor::redisplay_tempo (bool immediate_redraw) if (!session) { return; } - - compute_current_bbt_points (leftmost_frame, leftmost_frame + (nframes_t)(edit_packer.get_width() * frames_per_unit)); // redraw rulers and measures - - if (immediate_redraw) { - hide_measures (); - - if (current_bbt_points) { - draw_measures (); - } - - } else if (tempo_map_change_idle_handler_id < 0) { - - tempo_map_change_idle_handler_id = Glib::signal_idle().connect (mem_fun (*this, &Editor::redraw_measures)); + compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_frames()); // redraw rulers and measures + compute_current_bbt_points (leftmost_frame, leftmost_frame + current_page_frames()); + if (immediate_redraw) { + redraw_measures (); + } else { +#ifdef GTKOSX + redraw_measures (); +#else + Glib::signal_idle().connect (mem_fun (*this, &Editor::redraw_measures)); +#endif } - - update_tempo_based_rulers (); + update_tempo_based_rulers (); // redraw rulers and measures } void @@ -148,9 +145,9 @@ Editor::compute_current_bbt_points (nframes_t leftmost, nframes_t rightmost) session->bbt_time(rightmost, next_beat); if (previous_beat.beats > 1) { - previous_beat.beats -= 1; + previous_beat.beats -= 1; } else if (previous_beat.bars > 1) { - previous_beat.bars--; + previous_beat.bars--; previous_beat.beats += 1; } previous_beat.ticks = 0; @@ -162,11 +159,9 @@ Editor::compute_current_bbt_points (nframes_t leftmost, nframes_t rightmost) next_beat.beats = 1; } next_beat.ticks = 0; - - if (current_bbt_points) { - delete current_bbt_points; - current_bbt_points = 0; - } + + delete current_bbt_points; + current_bbt_points = 0; current_bbt_points = session->tempo_map().get_points (session->tempo_map().frame_time (previous_beat), session->tempo_map().frame_time (next_beat) + 1); } @@ -174,39 +169,30 @@ Editor::compute_current_bbt_points (nframes_t leftmost, nframes_t rightmost) void Editor::hide_measures () { - tempo_lines->hide(); - marker_tempo_lines->hide(); + if (tempo_lines) + tempo_lines->hide(); } bool Editor::redraw_measures () { - hide_measures (); draw_measures (); - tempo_map_change_idle_handler_id = -1; return false; } void Editor::draw_measures () { - if (session == 0 || _show_measures == false || + if (session == 0 || _show_measures == false || !current_bbt_points || current_bbt_points->empty()) { return; } - tempo_lines->draw(*current_bbt_points, frames_per_unit); - marker_tempo_lines->draw(*current_bbt_points, frames_per_unit); - - /*time_line_group->raise_to_top(); - time_line_group->lower(1);*/ - marker_time_line_group->raise_to_top(); - //marker_time_line_group->lower(1); - - /* the cursors are always on top of everything */ - cursor_group->raise_to_top(); + if (tempo_lines == 0) { + tempo_lines = new TempoLines(*track_canvas, time_line_group, physical_screen_height); + } - return; + tempo_lines->draw(*current_bbt_points, frames_per_unit); } void @@ -218,7 +204,7 @@ Editor::mouse_add_new_tempo_event (nframes64_t frame) TempoMap& map(session->tempo_map()); TempoDialog tempo_dialog (map, frame, _("add")); - + tempo_dialog.set_position (Gtk::WIN_POS_MOUSE); //this causes compiz to display no border. //tempo_dialog.signal_realize().connect (bind (sigc::ptr_fun (set_decoration), &tempo_dialog, Gdk::WMDecoration (Gdk::DECOR_BORDER|Gdk::DECOR_RESIZEH))); @@ -234,20 +220,20 @@ Editor::mouse_add_new_tempo_event (nframes64_t frame) double bpm = 0; BBT_Time requested; - + bpm = tempo_dialog.get_bpm (); double nt = tempo_dialog.get_note_type(); bpm = max (0.01, bpm); - + tempo_dialog.get_bbt_time (requested); - + begin_reversible_command (_("add tempo mark")); XMLNode &before = map.get_state(); map.add_tempo (Tempo (bpm,nt), requested); XMLNode &after = map.get_state(); session->add_command(new MementoCommand(map, &before, &after)); commit_reversible_command (); - + //map.dump (cerr); } @@ -264,11 +250,11 @@ Editor::mouse_add_new_meter_event (nframes64_t frame) meter_dialog.set_position (Gtk::WIN_POS_MOUSE); - //this causes compiz to display no border.. + //this causes compiz to display no border.. //meter_dialog.signal_realize().connect (bind (sigc::ptr_fun (set_decoration), &meter_dialog, Gdk::WMDecoration (Gdk::DECOR_BORDER|Gdk::DECOR_RESIZEH))); ensure_float (meter_dialog); - + switch (meter_dialog.run ()) { case RESPONSE_ACCEPT: break; @@ -278,7 +264,7 @@ Editor::mouse_add_new_meter_event (nframes64_t frame) double bpb = meter_dialog.get_bpb (); bpb = max (1.0, bpb); // XXX is this a reasonable limit? - + double note_type = meter_dialog.get_note_type (); BBT_Time requested; @@ -289,7 +275,7 @@ Editor::mouse_add_new_meter_event (nframes64_t frame) map.add_meter (Meter (bpb, note_type), requested); session->add_command(new MementoCommand(map, &before, &map.get_state())); commit_reversible_command (); - + //map.dump (cerr); } @@ -307,7 +293,7 @@ Editor::remove_tempo_marker (ArdourCanvas::Item* item) if ((tempo_marker = dynamic_cast (marker)) == 0) { fatal << _("programming error: marker for tempo is not a tempo marker!") << endmsg; /*NOTREACHED*/ - } + } if (tempo_marker->tempo().movable()) { Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::real_remove_tempo_marker), &tempo_marker->tempo())); @@ -332,7 +318,7 @@ Editor::edit_meter_section (MeterSection* section) double bpb = meter_dialog.get_bpb (); bpb = max (1.0, bpb); // XXX is this a reasonable limit? - + double note_type = meter_dialog.get_note_type (); begin_reversible_command (_("replace tempo mark")); @@ -351,7 +337,7 @@ Editor::edit_tempo_section (TempoSection* section) tempo_dialog.set_position (Gtk::WIN_POS_MOUSE); ensure_float (tempo_dialog); - + switch (tempo_dialog.run ()) { case RESPONSE_ACCEPT: break; @@ -364,16 +350,16 @@ Editor::edit_tempo_section (TempoSection* section) BBT_Time when; tempo_dialog.get_bbt_time(when); bpm = max (0.01, bpm); - + cerr << "Editing tempo section to be at " << when << endl; session->tempo_map().dump (cerr); begin_reversible_command (_("replace tempo mark")); - XMLNode &before = session->tempo_map().get_state(); + XMLNode &before = session->tempo_map().get_state(); session->tempo_map().replace_tempo (*section, Tempo (bpm,nt)); session->tempo_map().dump (cerr); session->tempo_map().move_tempo (*section, when); session->tempo_map().dump (cerr); - XMLNode &after = session->tempo_map().get_state(); + XMLNode &after = session->tempo_map().get_state(); session->add_command (new MementoCommand(session->tempo_map(), &before, &after)); commit_reversible_command (); } @@ -392,7 +378,7 @@ Editor::edit_tempo_marker (ArdourCanvas::Item *item) if ((tempo_marker = dynamic_cast (marker)) == 0) { fatal << _("programming error: marker for tempo is not a tempo marker!") << endmsg; /*NOTREACHED*/ - } + } edit_tempo_section (&tempo_marker->tempo()); } @@ -411,8 +397,8 @@ Editor::edit_meter_marker (ArdourCanvas::Item *item) if ((meter_marker = dynamic_cast (marker)) == 0) { fatal << _("programming error: marker for meter is not a meter marker!") << endmsg; /*NOTREACHED*/ - } - + } + edit_meter_section (&meter_marker->meter()); } @@ -443,7 +429,7 @@ Editor::remove_meter_marker (ArdourCanvas::Item* item) if ((meter_marker = dynamic_cast (marker)) == 0) { fatal << _("programming error: marker for meter is not a meter marker!") << endmsg; /*NOTREACHED*/ - } + } if (meter_marker->meter().movable()) { Glib::signal_idle().connect (bind (mem_fun(*this, &Editor::real_remove_meter_marker), &meter_marker->meter()));