Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

DEBUG ASSERT in CueControl::loadCuesFromTrack() #10478

Closed
mixxxbot opened this issue Aug 23, 2022 · 3 comments
Closed

DEBUG ASSERT in CueControl::loadCuesFromTrack() #10478

mixxxbot opened this issue Aug 23, 2022 · 3 comments
Milestone

Comments

@mixxxbot
Copy link
Collaborator

Reported by: uklotzde
Date: 2021-07-21T12:10:06Z
Status: Fix Committed
Importance: High
Launchpad Issue: lp1937074
Tags: cue


Happened when loading a track:

critical [CachingReaderWorker 1] DEBUG ASSERT: "mainCuePosition == mixxx::audio::kStartFramePos" in function void CueControl::loadCuesFromTrack() at ../src/engine/controls/cuecontrol.cpp:653

Unfortunately, I was not able to reproduce this assertion by reloading the track. Hope to find another one and then stop Mixxx before the database gets updated. But I guess this could also be a race condition.

@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2021-07-24T10:30:20Z


The debug assertion is wrong, this case needs to be handled for previously imported and played tracks.

Moreover the synchronization code creates unneeded cue points.

@mixxxbot
Copy link
Collaborator Author

Commented by: uklotzde
Date: 2021-07-24T10:43:49Z


#4137

@mixxxbot
Copy link
Collaborator Author

Issue closed with status Fix Committed.

@mixxxbot mixxxbot transferred this issue from another repository Aug 24, 2022
@mixxxbot mixxxbot added this to the 2.4.0 milestone Aug 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant