Skip to content

Conversation

aschackmull
Copy link
Contributor

This change currently has no effect, but will enable a new compiler warning in the future.

Copy link
Contributor

@ginsbach ginsbach left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To confirm that you did not miss any predicates, I compiled all cpp queries with my warning-enabled draft branch. Your changes make all warnings disappear.

LGTM

@aschackmull aschackmull merged commit d79a7e8 into github:main Oct 13, 2022
@aschackmull aschackmull deleted the dataflow/additional branch October 13, 2022 11:02
owen-mc added a commit to owen-mc/codeql that referenced this pull request Nov 29, 2022
owen-mc added a commit to owen-mc/codeql that referenced this pull request Nov 29, 2022
owen-mc added a commit to owen-mc/codeql that referenced this pull request Nov 29, 2022
owen-mc added a commit to owen-mc/codeql that referenced this pull request Nov 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants