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

Tweaks to iOS examples to remove ofxAccelerometer and ofxiOSAccelerometer #7371

Merged
merged 6 commits into from
Mar 9, 2023

Conversation

artificiel
Copy link
Contributor

@artificiel artificiel commented Mar 9, 2023

Tweaks to iOS examples referencing ofxAccelerometer. All such examples now run (some ARC adjustments done) confirmed by a quick test (the app loads and does not crash). Removed references in xcconfig and some scripts also.

The ofxAccelerometer addon itself is not removed as android still refers to it -- some androidian should check it out and if still required, move it into ofxAndroid?

Tweaks to iOS CoreMotion example
removal of ofxIOSAccelerometer
All touched examples quickly tweaked for ARC as needed and tested on a device (confirmed that they don't crash; not validated in functionality).
@artificiel artificiel changed the title Tweaks to iOS CoreMotion example (now compiles) Tweaks to iOS examples to remove ofxAccelerometer and ofxiOSAccelermeter Mar 9, 2023
@artificiel artificiel changed the title Tweaks to iOS examples to remove ofxAccelerometer and ofxiOSAccelermeter Tweaks to iOS examples to remove ofxAccelerometer and ofxiOSAccelerometer Mar 9, 2023
@ofTheo ofTheo merged commit f4193b8 into openframeworks:master Mar 9, 2023
@ofTheo
Copy link
Member

ofTheo commented Mar 9, 2023

Awesome!! Thanks!

@artificiel artificiel deleted the ios-coremotion-example branch March 9, 2023 17:09
dimitre pushed a commit to dimitre/openFrameworks that referenced this pull request Mar 18, 2023
@artificiel artificiel mentioned this pull request Nov 1, 2023
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.

2 participants