Skip to content

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