-
Notifications
You must be signed in to change notification settings - Fork 118
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
Add new API call OfxSetHost #53
Conversation
Adds OfxSetHost and a few doc notes
host's instead of your
include/ofxCore.h
Outdated
* This host call is not specified in earlier implementation of the API. | ||
* Therefore host must check if the plugin implemented it and not assume symbol exists. | ||
* The order of calls is then: 1) OfxGetHost, 2) OfxGetNumberOfPlugins, 3) OfxGetPlugin | ||
* This pointer is only assumed valid until OfxGetPlugin where it might get reset. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The host pointer
Fixed comments as per GaryO's suggestions
Fixed a note for OfxGetHost
The documentation has not been updated for this change. However, rather than make this take even longer to merge, I'll update the doc as a separate commit after merging this shortly. |
See standard-change proposal at #71 |
Note that this has not been merged into master -- only onto a feature branch, pending resolution of open questions and final approval. |
Adds OfxSetHost and a few doc notes
Please read the contribution guidelines.
Description
Please provide a description of what this PR is meant to fix, and link it to the corresponding open issue with its tag.
Also explain how it works (if it's not going to be very clear from the code).