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

Synchronize all property access to a single thread #157

Open
nicolaspayette opened this issue May 5, 2015 · 0 comments
Open

Synchronize all property access to a single thread #157

nicolaspayette opened this issue May 5, 2015 · 0 comments

Comments

@nicolaspayette
Copy link
Member

Timing issues between things happening from code and things happening from the GUI are delicate. #156 is a good example of that.

As discussed with @qiemem on Gitter (https://gitter.im/CRESS-Surrey/eXtraWidgets?at=5548ec8952bceea22c3813dd):

All these problems stem from the fact that the widget models (i.e., Reader/Writer) can be accessed from both the job thread and the awt event thread. I didn't think of that before, but what if every call to reader/writer was blocking and ran: on the job thread if running headless; on the event thread otherwise. Since the job thread is allowed to wait on the event thread and that the opposite would never happen, would wouldn't get locking. And we'd have the garantee that everything always happens in the right order.
That's not a trivial change, however...

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