-
Notifications
You must be signed in to change notification settings - Fork 182
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
Implicit include of checked header files #440
Conversation
…iles plus the fix to push/pop checked scopes.
Referencing #431 again. Is this ready for us to test (in combination with checkedc/checkedc-clang#998)? |
@mattmccutchen-cci Yes, it is. You will need the |
may not be present in all compilation environments (Ex. Windows). Therefore, the wrapper header files and the Checked C header files should guard their inclusion using __has_include_next. This will also cause changes in the expected output of some test cases.
…ed.h and inttypes_checked.h use the type wchar_t, whose definition is available in stddef.h. The file inttypes_checked.h includes stddef.h but the other two don't.
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.
Looks good, thanks.
Header file changes for the implicit inclusion of checked headers. This PR also deprecates the use of the macro
__cplusplus
in the header files and uses the new macro__checkedc
instead.