ardour icon indicating copy to clipboard operation
ardour copied to clipboard

unlimited scrolling on pianoroll caused fpe

Open ferprojo opened this issue 5 months ago • 1 comments

Found that when I accidentally scrolled too much on the piano roll window, it triggered a floating point exception. I looked at how the Editor implemented set_samples_per_pixel, and did the same, just returning when spp < 1. I'm new to ardour development so I don't know if this is correct, but it fixed it for me.

ferprojo avatar Aug 07 '25 03:08 ferprojo

I'd assume the better solution would be if (spp < 1) { spp = 1; } Then again ideally that case should not happen in the first place, and we just add an assert (spp > 0);

x42 avatar Sep 28 '25 11:09 x42