Skip to content

C++: Add tests for copy constructor calls with implied source#8592

Merged
MathiasVP merged 2 commits into
github:mainfrom
jketema:implied-cctor-source
Apr 7, 2022
Merged

C++: Add tests for copy constructor calls with implied source#8592
MathiasVP merged 2 commits into
github:mainfrom
jketema:implied-cctor-source

Conversation

@jketema

@jketema jketema commented Mar 29, 2022

Copy link
Copy Markdown
Contributor

No description provided.

@jketema jketema added the depends on internal PR This PR should only be merged in sync with an internal Semmle PR label Mar 29, 2022
@jketema jketema force-pushed the implied-cctor-source branch from 14d6504 to e91c042 Compare April 4, 2022 10:48
@jketema jketema marked this pull request as ready for review April 4, 2022 10:59
@jketema jketema requested a review from a team as a code owner April 4, 2022 10:59

@geoffw0 geoffw0 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

👍

@MathiasVP MathiasVP merged commit d4834cb into github:main Apr 7, 2022
jketema added a commit to jketema/codeql that referenced this pull request Apr 7, 2022
…rce"

This reverts commit d4834cb, reversing
changes made to 268a3fd.
MathiasVP added a commit that referenced this pull request Apr 7, 2022
Revert "Merge pull request #8592 from jketema/implied-cctor-source"
jketema added a commit to jketema/codeql that referenced this pull request Apr 7, 2022
jketema added a commit that referenced this pull request Apr 7, 2022
Revert "Revert "Merge pull request #8592 from jketema/implied-cctor-source""
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 documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants