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.

Sign up for GitHub

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

Jump to bottom

Fix presumed bug in d0f475d (min should be max). Fixes #1842 #1843

Closed
wants to merge 1 commit into from
Closed

Fix presumed bug in d0f475d (min should be max). Fixes #1842 #1843

wants to merge 1 commit into from

Conversation

danstowell
Copy link
Member

This caused lang freeze on ARM Cortex A8 devices, presumably because they are single-concurrency and the bug led to a thread pool of size zero.

@danstowell
Fix presumed bug in d0f475d (min should be max). Fixes supercollider#…
d17ecb8 
…1842

This caused lang freeze on ARM Cortex A8 devices, presumably
because they are single-concurrency and the bug led to a thread pool of
size zero.
@danstowell danstowell added this to the 3.8 milestone Jan 31, 2016
@danstowell danstowell added architecture: arm comp: sclang sclang C++ implementation (primitives, etc.). for changes to class lib use "comp: class library" labels Jan 31, 2016
@timblechmann
Copy link
Contributor

probably not sufficient. later a division by zero is going to happen, if hardware_concurrency is not implemented.

compare #1844

@danstowell
Copy link
Member Author

superceded by #1844

@danstowell danstowell closed this Jan 31, 2016
@danstowell danstowell deleted the fix_d43a493_singlethread branch January 31, 2016 20:13
@danstowell danstowell mentioned this pull request Jan 31, 2016
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
architecture: arm comp: sclang sclang C++ implementation (primitives, etc.). for changes to class lib use "comp: class library"
Projects
None yet
Milestone
3.8
Development

Successfully merging this pull request may close these issues.
2 participants
@danstowell @timblechmann