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.

Sign up for GitHub

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

Jump to bottom

NP_GetPluginVersion has incorrect const-ness #13

Open

GoogleCodeExporter opened this issue Sep 1, 2015 · 1 comment

Labels

auto-migrated Priority-Medium Type-Defect

GoogleCodeExporter commented Sep 1, 2015

Fairly minor thing; I can't find any plugin out there that exports this, short 
of demo ones. But may as well correct it to match NP_GetMIMEDescription's fix. 
Mozilla's already casting the function to the const version anyway.

http://mxr.mozilla.org/mozilla-central/source/dom/plugins/base/nsPluginsDirUnix.
cpp#359

Patch attached.

Original issue reported on code.google.com by davidben@davidben.net on 9 Oct 2011 at 5:31

Attachments:

0001-Fix-const-ness-of-NP_GetPluginVersion.patch

The text was updated successfully, but these errors were encountered:

Author

GoogleCodeExporter commented Sep 1, 2015

I support. Pointers to const should be used everywhere, where data is not 
intended to be modified. An exception might be a pointer to const pointer to 
const data, which is not supported in C.

Original comment by Van...@gmail.com on 23 Feb 2012 at 9:40

GoogleCodeExporter added Priority-Medium Type-Defect auto-migrated labels

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