ofxIlda icon indicating copy to clipboard operation
ofxIlda copied to clipboard
verified publisher icon memo

small fixes for openFrameworks 0.11.2

Open hiroyuki opened this issue 3 years ago • 4 comments

  • replaced ofPoint to glm::vec3
  • change testApp to ofApp
  • and test compile basic example ( need to change file name above )

warning: ofxIlda example RenderTarget still needs to be fixed.

hiroyuki avatar Oct 17 '22 09:10 hiroyuki

Hi, thx so much for doing this. is ofPoint deprecated? I would have thought, for future compatibility, it would make more sense to keep an OF type (such as ofPoint, or ofVec2f, or ofVec3f) - even if internally typedef'ed to glm:vec3, just so that in the future, if glm is replaced, older codebases would carry on working?

memoakten avatar Oct 17 '22 18:10 memoakten

Hi, Thanks for your great addon! Yes, I agree it would be great if it works with OF type and your concern about the future replacement of glm. I just thought it is better to keep your nice addon straightforward to use. Especially for people unfamiliar with old OF syntax, it can be hard to fix them by themselves.

hiroyuki avatar Oct 18 '22 13:10 hiroyuki

oh but does it currently not work? has ofPoint been removed? or ofVec2f?

memoakten avatar Oct 19 '22 03:10 memoakten

No it's not working. The ofPoint is not removed but the interfaces of oF functions replaced it with glm mostly.

hiroyuki avatar Oct 20 '22 05:10 hiroyuki