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

Processing order of late events is undefined #337

Open
olofson opened this issue Dec 7, 2017 · 2 comments
Open

Processing order of late events is undefined #337

olofson opened this issue Dec 7, 2017 · 2 comments

Comments

@olofson
Copy link
Owner

olofson commented Dec 7, 2017

It seems like the "late API message" mechanism (which just clamps the timestamp to the start of the current frame) results in undefined processing order, and that's why we're having problems like olofson/koboredux#401 and olofson/koboredux#512.

@olofson
Copy link
Owner Author

olofson commented Dec 7, 2017

I don't think this matters, actually - or rather, avoiding this issue does not solve the problem. After some experiments with this, I've concluded that the aforementioned Kobo Redux problems can occur even if messages arrive in order.

It seems like the problem is with creating or finding the A2_TNEWVOICE placeholder handles, which are to serve as event queues for voices not yet instantiated.

@olofson olofson modified the milestones: v1.9.3, v1.9.4 Dec 9, 2017
@olofson
Copy link
Owner Author

olofson commented Dec 9, 2017

Turns out the Kobo Redux problems were eliminated by avoiding dropped messages due to API buffer overflows, so I'm leaving this for the next release.

@olofson olofson modified the milestones: v1.9.4, v1.9.5+ Dec 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant