beatloop rolls occasionally reset the playhead to the start

Bug #1362394 reported by Owen Williams
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mixxx
Fix Released
Critical
Owen Williams

Bug Description

This is a really hard-to-reproduce party stopper: Every so often a beatloop roll will cause the track to seek to the beginning and stop completely. I'm not sure, but I think it might happen when the loop is disabled, not when the loop is active.

Revision history for this message
Owen Williams (ywwg) wrote :

it's probably some sort of race with slipenabled and slipposition getting all messed up.

Revision history for this message
Owen Williams (ywwg) wrote :
Revision history for this message
Owen Williams (ywwg) wrote :

My best theory for this bug:

in slotControlSlip, the enabled value was set before the toggled value. If the timing was just wrong, processSlip could be executed just between those two calls. Perhaps this could cause m_dSlipPosition to not be set to the current playposition, so when slip was released it would rewind to the beginning. But I'm not sure that's right, because playback would also stop, and there's nothing in this codepath that would cause playback to stop. Also I'd have thought that Toggled would be on for the next loop, so it would still reset the slip position.

So I'm going to leave this bug open for a little while and really hammer the rolling loops in my testing :)

Revision history for this message
Owen Williams (ywwg) wrote :

erm, I mean when I see this bug, playback stops, but with this theory I don't know why the track would stop.

Revision history for this message
Owen Williams (ywwg) wrote :

I haven't seen this in a while, so I'm fairly sure it's fixed.

Revision history for this message
RJ Skerry-Ryan (rryan) wrote :

let's re-open if it returns

Changed in mixxx:
assignee: nobody → Owen Williams (ywwg)
status: New → Fix Committed
RJ Skerry-Ryan (rryan)
Changed in mixxx:
status: Fix Committed → Fix Released
Revision history for this message
Swiftb0y (swiftb0y) wrote :

Mixxx now uses GitHub for bug tracking. This bug has been migrated to:
https://github.com/mixxxdj/mixxx/issues/7565

lock status: Metadata changes locked and limited to project staff
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.