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

[critical bug fix] Osc scope2 #7981

Merged
merged 3 commits into from
May 22, 2024
Merged

Conversation

artificiel
Copy link
Contributor

replaces #7978 which had a garbled commit

@artificiel artificiel changed the title Osc scope2 [critical bug fix] Osc scope2 May 21, 2024
@NickHardeman
Copy link
Contributor

Small thing, but is this commit needed for this PR?
7109a97

@artificiel
Copy link
Contributor Author

yes I noticed in the GitHub actions some platforms were failing because of missing include (although macOS somehow has it without the explicit include)

@NickHardeman
Copy link
Contributor

Just tested on Windows 10 and VS 2022 using the StressofxOscReceiver test and did not encounter any issues. :)

@NickHardeman
Copy link
Contributor

When I add a ofxOscReceiver rc; to the setup() function, it prints "ready to roll" but hangs. Is this an issue that is addressed in another PR?

#ifdef INSTANTLY_OUT_OF_SCOPE
	ofLogNotice("will hang");
	ofxOscReceiver r;
	ofLogNotice("will hang if thread problem");
	std::unique_ptr<ofxOscReceiver> r = std::make_unique<ofxOscReceiver>();
	r = std::make_unique<ofxOscReceiver>();

	ofxOscReceiver rc;
#endif
	ofLogNotice("ready to roll");

@artificiel
Copy link
Contributor Author

is that on windows? if so, please try #7979

if not I will try to reproduce.

@NickHardeman
Copy link
Contributor

Yeah on Windows; #7979 fixes it!

@artificiel
Copy link
Contributor Author

phew! thanks for pulling this together it was a bit tricky to get all parts.

note that once this (#7981 etc) is merged, #7925 should then pass all tests, and contains a ofxTest that has a case for this recent thread stuff.

@ofTheo ofTheo merged commit bfbedaf into openframeworks:master May 22, 2024
11 checks passed
@ofTheo
Copy link
Member

ofTheo commented May 22, 2024

Thanks @artificiel @NickHardeman !!!!

@artificiel artificiel deleted the osc-scope2 branch May 22, 2024 03:14
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.

None yet

3 participants