double theoretical_end = ext.second;
/* the summary should encompass the full extent of everywhere we've visited since the session was opened */
- if ( _leftmost < theoretical_start)
+ if (_leftmost < theoretical_start)
theoretical_start = _leftmost;
- if ( _rightmost > theoretical_end )
+ if (_rightmost > theoretical_end)
theoretical_end = _rightmost;
/* range-check */
/* paint a non-bg colored strip to represent the track itself */
- if ( _track_height > 4 ) {
+ if (_track_height > 4) {
cairo_set_source_rgb (cr, 0.2, 0.2, 0.2);
cairo_set_line_width (cr, _track_height - 1);
cairo_move_to (cr, 0, y + _track_height / 2);
cairo_line_to (cr, get_width(), y + _track_height / 2);
cairo_stroke (cr);
}
-
+
StreamView* s = (*i)->view ();
if (s) {
cairo_set_line_width (cr, _track_height * 0.8);
s->foreach_regionview (sigc::bind (
- sigc::mem_fun (*this, &EditorSummary::render_region),
- cr,
- y + _track_height / 2
- ));
+ sigc::mem_fun (*this, &EditorSummary::render_region),
+ cr,
+ y + _track_height / 2
+ ));
}
y += _track_height;
/* maintain the leftmost and rightmost locations that we've ever reached */
samplecnt_t const leftmost = _editor->leftmost_sample ();
- if ( leftmost < _leftmost) {
+ if (leftmost < _leftmost) {
_leftmost = leftmost;
_background_dirty = true;
}
samplecnt_t const rightmost = leftmost + _editor->current_page_samples();
- if ( rightmost > _rightmost) {
+ if (rightmost > _rightmost) {
_rightmost = rightmost;
_background_dirty = true;
}
- //draw the background (regions, markers, etc ) if they've changed
+ /* draw the background (regions, markers, etc) if they've changed */
if (!_image || _background_dirty) {
render_background_image ();
_background_dirty = false;
void
EditorSummary::render_region (RegionView* r, cairo_t* cr, double y) const
{
+ /*NOTE: you can optimize this operation by coalescing adjacent regions into a single line stroke.
+ * In a session with a single track ~1,000 regions, this reduced render time from 14ms to 11 ms.
+ * However, you lose a lot of visual information. The current method preserves a sense of separation between regions.
+ * The current method shows the current selection (red regions), which needs to be preserved if this is optimized.
+ * I think it's not worth it for now, but we might choose to revisit this someday.
+ */
+
uint32_t const c = r->get_fill_color ();
cairo_set_source_rgb (cr, UINT_RGBA_R (c) / 255.0, UINT_RGBA_G (c) / 255.0, UINT_RGBA_B (c) / 255.0);
{
_old_follow_playhead = _editor->follow_playhead ();
- if (ev->button == 3) { //right-click: show the reset menu action
+ if (ev->button == 3) { // right-click: show the reset menu action
using namespace Gtk::Menu_Helpers;
Gtk::Menu* m = manage (new Gtk::Menu);
MenuList& items = m->items ();
_editor->set_follow_playhead (false);
_move_dragging = true;
-
+
_last_mx = ev->x;
_last_my = ev->y;
_last_dx = 0;
_last_y_delta = 0;
get_window()->set_cursor (*_editor->_cursors->expand_left_right);
-
+
}
return true;
void
EditorSummary::reset_to_extents()
{
- //reset as if the user never went anywhere outside the extents
+ /* reset as if the user never went anywhere outside the extents */
_leftmost = max_samplepos;
_rightmost = 0;
}
void
-EditorSummary::summary_zoom_step ( int steps /* positive steps to zoom "out" , negative steps to zoom "in" */ )
+EditorSummary::summary_zoom_step (int steps /* positive steps to zoom "out" , negative steps to zoom "in" */ )
{
pair<double, double> xn;
xn.first -= steps;
xn.second += steps;
- //for now, disallow really close zooming-in from the scroomer. ( currently it causes the start-offset to 'walk' because of integer limitations. to fix this, probably need to maintain float throught the get/set_editor() path )
+ /* for now, disallow really close zooming-in from the scroomer. (Currently it
+ * causes the start-offset to 'walk' because of integer limitations.
+ * To fix this, probably need to maintain float throught the get/set_editor() path.)
+ */
if (steps<0) {
- if ( (xn.second-xn.first) < 2)
+ if ((xn.second - xn.first) < 2)
return;
}
{
if (_move_dragging) {
- //To avoid accidental zooming, the mouse must move exactly vertical, not diagonal, to trigger a zoom step
- //we use screen coordinates for this, not canvas-based grab_x
+ /* To avoid accidental zooming, the mouse must move exactly vertical, not diagonal, to trigger a zoom step
+ * we use screen coordinates for this, not canvas-based grab_x */
double mx = ev->x;
double dx = mx - _last_mx;
double my = ev->y;
double dy = my - _last_my;
- //do zooming in windowed "steps" so it feels more reversible ?
+ /* do zooming in windowed "steps" so it feels more reversible ? */
const int stepsize = 2;
int y_delta = _start_mouse_y - my;
y_delta = y_delta / stepsize;
- //do the zoom?
+ /* do the zoom? */
const float zscale = 3;
- if ( (dx==0) && (_last_dx ==0) && (y_delta != _last_y_delta) ) {
+ if ((dx == 0) && (_last_dx == 0) && (y_delta != _last_y_delta)) {
- summary_zoom_step( dy * zscale );
+ summary_zoom_step (dy * zscale);
- //after the zoom we must re-calculate x-pos grabs
+ /* after the zoom we must re-calculate x-pos grabs */
pair<double, double> xr;
get_editor (&xr);
_start_editor_x = xr;
_start_mouse_x = ev->x;
-
+
_last_y_delta = y_delta;
}
-
- //always track horizontal movement, if any
- if ( dx != 0 ) {
+
+ /* always track horizontal movement, if any */
+ if (dx != 0) {
double x = _start_editor_x.first;
x += ev->x - _start_mouse_x;
-
+
if (x < 0) {
x = 0;
}
- //zoom-behavior-tweaks
- //protect the right edge from expanding beyond the end
+ /* zoom-behavior-tweaks: protect the right edge from expanding beyond the end */
pair<double, double> xr;
get_editor (&xr);
double w = xr.second - xr.first;
- if ( x + w < get_width() ) {
+ if (x + w < get_width()) {
set_editor (x);
}
}
xr.first += dx;
} else if (_zoom_trim_position == RIGHT) {
- //zoom-behavior-tweaks
- //protect the right edge from expanding beyond the edge
- if ( (xr.second + dx) < get_width() ) {
+ /* zoom-behavior-tweaks: protect the right edge from expanding beyond the edge */
+ if ((xr.second + dx) < get_width()) {
xr.second += dx;
}
set_editor (xr);
} else {
- set_cursor ( get_position(ev->x, ev->y) );
+ set_cursor (get_position (ev->x, ev->y));
}
return true;
switch (ev->direction) {
case GDK_SCROLL_UP: {
-
- summary_zoom_step( -4 );
-
+
+ summary_zoom_step (-4);
+
return true;
} break;
-
+
case GDK_SCROLL_DOWN: {
-
- summary_zoom_step( 4 );
-
+
+ summary_zoom_step (4);
+
return true;
} break;
-
+
case GDK_SCROLL_LEFT:
if (Keyboard::modifier_state_equals (ev->state, Keyboard::ScrollZoomHorizontalModifier)) {
_editor->temporal_zoom_step (false);
projects / ardour.git / blobdiff