Skip to content

C++: built-in type tweaks #18

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 31, 2018

Conversation

nickrolfe
Copy link
Contributor

Two changes that go hand-in-hand with internal extractor improvements, related to handling of built-in types:

  1. Properly formatted (and complete) comments for each built-in type kind.
  2. A test change to match the new behaviour that the error and unknown types have size 1, not 0.

@nickrolfe nickrolfe added C++ depends on internal PR This PR should only be merged in sync with an internal Semmle PR labels Aug 6, 2018
@nickrolfe nickrolfe force-pushed the builtin_types_on_demand3 branch from 26b72b8 to d4f9b5e Compare August 31, 2018 11:08
@ian-semmle ian-semmle merged commit eac82f2 into github:master Aug 31, 2018
@nickrolfe nickrolfe deleted the builtin_types_on_demand3 branch August 31, 2018 15:41
tamasvajk referenced this pull request in tamasvajk/codeql Sep 16, 2021
Kotlin: Add a compilations consistency query
aibaars pushed a commit that referenced this pull request Oct 14, 2021
Don't generate an index for optional fields that occur at most once
erik-krogh referenced this pull request in erik-krogh/ql Dec 15, 2021
implement `ql/override-swapped-name`
erik-krogh referenced this pull request in erik-krogh/ql Dec 15, 2021
implement `ql/override-swapped-name`
RasmusWL pushed a commit that referenced this pull request Mar 30, 2023
Initial PR - Proof of Concept using CNG
dbartol pushed a commit that referenced this pull request Dec 18, 2024
feat(triggers): New query and support for trigger-based severity decisions
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