smokephil/ardour
1
0
Fork 0
mirror of https://github.com/Ardour/ardour.git synced 2025-12-06 14:54:56 +01:00
Code Issues Projects Releases Wiki Activity

CueEditor: fix a corner case when displaying a ridiculously short region

Browse source 
If there are less samples to show than there are pixels to show it, samples per pixel would
be zero. Limit it to spp = 1
This commit is contained in:
Paul Davis 2025-11-19 11:08:18 -07:00
parent 8cb3f79e0f
commit 1e430325d0
1 changed files with 14 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

15
gtk2_ardour/editing_context.cc
View file

@ -2613,7 +2613,18 @@ EditingContext::reset_zoom (samplecnt_t spp)
}
std::pair<timepos_t, timepos_t> ext = max_zoom_extent();
samplecnt_t max_extents_pp = max_extents_scale() * ((ext.second.samples() - ext.first.samples()) / _track_canvas_width);
samplecnt_t p = (ext.second.samples() - ext.first.samples()) / _track_canvas_width;
if (spp == 0) {
/* less samples than pixels */
spp = 1;
}
if (p == 0) {
/* Less samples to display than there are pixels, use spp = 1 */
p = 1;
}
samplecnt_t max_extents_pp = max_extents_scale() * p;
if (spp > max_extents_pp) {
spp = max_extents_pp;
@ -2623,6 +2634,8 @@ EditingContext::reset_zoom (samplecnt_t spp)
return;
}
assert (spp != 0);
pending_visual_change.add (VisualChange::ZoomLevel);
pending_visual_change.samples_per_pixel = spp;
ensure_visual_change_idle_handler ();