Skip to content

C++: test for __declspec(guard(...)) #24

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

Merged
merged 1 commit into from
Aug 28, 2018

Conversation

nickrolfe
Copy link
Contributor

We see projects in the wild using __declspec(guard(...)) with MSVC. This PR adds a test to show they don't cause any errors; it depends on a corresponding change in the extractor.

@nickrolfe nickrolfe added the C++ label Aug 7, 2018
jonas-semmle pushed a commit that referenced this pull request Aug 7, 2018
JavaScript: Minor library work.
@nickrolfe nickrolfe added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Aug 9, 2018
@ian-semmle ian-semmle merged commit 1d202dd into github:master Aug 28, 2018
@nickrolfe nickrolfe deleted the declspec_guard branch August 28, 2018 15:16
aibaars added a commit that referenced this pull request Oct 14, 2021
Add buffered writing and gzip compression for trap files
smowton pushed a commit to smowton/codeql that referenced this pull request Oct 28, 2021
erik-krogh pushed a commit to erik-krogh/ql that referenced this pull request Dec 15, 2021
erik-krogh pushed a commit to erik-krogh/ql that referenced this pull request Dec 15, 2021
dbartol pushed a commit that referenced this pull request Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C++ depends on internal PR This PR should only be merged in sync with an internal Semmle PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants