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

Mark functions from c includes with noexception #84

Merged
merged 1 commit into from
Mar 13, 2016

Conversation

osialr
Copy link
Contributor

@osialr osialr commented Mar 11, 2016

This allows the Generator to avoid wrapping the function invocations with
try/catch when no exception is possible

@saudet
Copy link
Member

saudet commented Mar 12, 2016

GCC doesn't do these kinds of optimization by default because we might have callbacks that throw C++ exceptions, but I think it's a good idea to offer the option to the user, and this looks like a good default.

Could you only remove the '\n' and replace that with a white space like it's done for other annotations? And I'll merge that in. Thanks!

This allows the Generator to avoid wrapping the function invocations with
try/catch when no exception is possible
@osialr osialr force-pushed the staging/mark-c-funcs-as-noexception branch from 33e1eda to 9086541 Compare March 12, 2016 16:10
@osialr
Copy link
Contributor Author

osialr commented Mar 12, 2016

Amended with your suggestions

saudet added a commit that referenced this pull request Mar 13, 2016
 * Annotate functions originating from `cinclude` with `@NoException`
@saudet saudet merged commit 079d3d6 into bytedeco:master Mar 13, 2016
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.

None yet

2 participants