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

Fix corruption crash when zooming on a large sample #7577

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

khoidauminh
Copy link
Contributor

Fixes #7576

Bug is very weird, so is PR.

I'd like to hear if there is a more solid fix.

@sakertooth
Copy link
Contributor

Doesn't even crash in debug mode...

@khoidauminh
Copy link
Contributor Author

@sakertooth When compiled with Release or ReleaseWithInfo I see crashes when the vectors de-allocate. I still see those vectors are unnecessary and add both overhead and memory cost.

Should we remove them, or should we wait til the sample cache/database PR gets merged and switch to the thumbnails at once?

@sakertooth
Copy link
Contributor

I'm going to submit a PR that basically starts the implementation of SampleWaveform::visualize at a clean slate. I've tried to make it as simple as possible and not worry about optimizations, considering the work me and you are doing with sample thumbnails. It removes all the vector allocations and only uses a single loop.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"double free or corruption" when trying to zoom in Song Editor, when there's a large sample loaded
2 participants