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

C API wrapper macros [NOT FOR MERGE] #4409

Closed
wants to merge 1 commit into from

Conversation

eric-hughes-tiledb
Copy link
Contributor

The CAPI_XXX_BEGIN and CAPI_XXX_END macros replace the explicit definitions of C API function with macro-generated text. This is principally to make available the function name as a static string available to the exception wrapper.

This PR is draft-only and should not be merged. It intended for (1) early review comments and (2) a test CI run. This version does not actually implement a complete system for the C API hook itself. It's primarily target is macros to define C API functions that can (later) add information required to log C API calls. The reason to use macros is to have the ability to using the stringify operator # on the function name; the name does not appear as text anywhere otherwise.


TYPE: NO_HISTORY
DESC: C API wrapper macros

The `CAPI_XXX_BEGIN` and `CAPI_XXX_END` macros replace the explicit definitions of C API function with macro-generated text. This is principally to make available the function name as a static string available to the exception wrapper.
@eric-hughes-tiledb eric-hughes-tiledb deleted the eh/wip/capi_hook/3 branch October 18, 2023 13:31
@eric-hughes-tiledb
Copy link
Contributor Author

Final version in #4430.

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.

1 participant