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

Please add NPAPI_VERSION_REVISION to provide true API version number #10

Open
GoogleCodeExporter opened this issue Mar 19, 2015 · 2 comments

Comments

@GoogleCodeExporter
Copy link

Whereas Issue#9 presented the fact that NPAPI doesn't have an actual API 
version number but instead has an ABI version number, and

Whereas Issue#9 presented the need for an actual API version number, and

Whereas the lack of intention for source-level compatibility to date does not 
mean it isn't worthy goal,

Be it resolved that an additional #define be added to the npapi-sdk, which can 
be combined along with the ABI version represented by NPAPI_VERSION_MAJOR + 
NPAPI_VERSION_MINOR , such as "NPAPI_VERSION_REVISION" , to create an actual 
API version that can be checked for and relied upon within plugin code.

Helper #defines, to support a simple #if statement to check required version 
info in plugin codes, would also be very nice.

Adding this new revision number to the versioning scheme in the pkg-config file 
would also be very helpful.


Original issue reported on code.google.com by i.staken...@gmail.com on 6 Sep 2011 at 7:48

@GoogleCodeExporter
Copy link
Author

To give a further rationale for this -- for the freewrl plugin I help maintain, 
I want to be able to use a particular #if statement in my code, based on 
version numbering, so that i define either a "char* NP_GetMIMEDescription()" or 
a "const char* NP_GetMIMEDescription()" so that whichever npapi.h et al a 
particular distro maintainer is going to use, the code will work appropriately.

(Freewrl does not embed npapi.h et all into its source distribution, nor does 
it intend to in the future)

Original comment by i.staken...@gmail.com on 7 Sep 2011 at 7:27

@GoogleCodeExporter
Copy link
Author

Issue 14 has been merged into this issue.

Original comment by stuart.morgan on 14 Nov 2011 at 6:25

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant