EmotiBit / ofxEmotiBit

OpenFrameworks library and examples for the EmotiBit
MIT License
33 stars 7 forks source link

Fix discovered emotibits mutex #232

Closed produceconsumerobot closed 1 month ago

produceconsumerobot commented 4 months ago

Description

Attempt to fix read access violation with _discoveredEmotiBits https://github.com/EmotiBit/ofxEmotiBit/issues/231

Requirements

Issues Referenced

Documentation update

Link to a change in documentation (if any)

Notes for Reviewer

List of new distributable files

none

Testing

Results

Feature Tests

none (it's possible we could add a stability test at some point, but it get's a bit laborious and under-defined)

Steps to test

See Testing above

Shared files

none

Checklist to allow merge

Screenshots:

nitin710 commented 3 months ago

Fix to PR description:

nitin710 commented 3 months ago

Testing Build

Build OS Build result
windows ✔️
mac ✔️
linux
nitin710 commented 3 months ago

Screenshot_2024-03-24_14-04-02

Tried adding path to file in config.make, but that did not resolve the error.

nitin710 commented 3 months ago

This PR is already merged into dev. Will be merged to master once build on linux is resolved.

nitin710 commented 1 month ago

LSL output was merged in master. See #239 .

This was an "intermediate" PR that was merged into dev as a part of a fast tracked release process. Consequently, it is now outdated and being closed.