Skip to content

[cpg] QT-676: Migrate parameter index to Option[Int]#6002

Merged
SuperUserDone merged 3 commits into
masterfrom
louis/QT-676
May 28, 2026
Merged

[cpg] QT-676: Migrate parameter index to Option[Int]#6002
SuperUserDone merged 3 commits into
masterfrom
louis/QT-676

Conversation

@SuperUserDone

@SuperUserDone SuperUserDone commented May 26, 2026

Copy link
Copy Markdown
Contributor

@SuperUserDone SuperUserDone requested a review from ml86 May 26, 2026 07:45
ml86
ml86 requested changes May 26, 2026
View reviewed changes
Comment thread ...n-cli/frontends/c2cpg/src/main/scala/io/joern/c2cpg/astcreation/AstForFunctionsCreator.scala Outdated
Comment thread .../csharpsrc2cpg/src/test/scala/io/joern/csharpsrc2cpg/querying/ast/ExtensionMethodTests.scala Outdated
Comment thread ...s/csharpsrc2cpg/src/test/scala/io/joern/csharpsrc2cpg/querying/ast/PropertySetterTests.scala Outdated
Comment thread ...s/csharpsrc2cpg/src/test/scala/io/joern/csharpsrc2cpg/querying/ast/PropertySetterTests.scala Outdated
Comment thread ...rontends/kotlin2cpg/src/test/scala/io/joern/kotlin2cpg/querying/CallableReferenceTests.scala Outdated
Comment thread ...nds/rubysrc2cpg/src/main/scala/io/joern/rubysrc2cpg/astcreation/AstForFunctionsCreator.scala Outdated
Comment thread ...nds/rubysrc2cpg/src/main/scala/io/joern/rubysrc2cpg/astcreation/AstForFunctionsCreator.scala Outdated
Comment thread ...in/scala/io/shiftleft/semanticcpg/language/types/structure/MethodParameterOutTraversal.scala Outdated
@SuperUserDone SuperUserDone marked this pull request as ready for review May 27, 2026 15:22
@SuperUserDone SuperUserDone merged commit 896ce6a into master May 28, 2026
8 checks passed
@ml86 ml86 mentioned this pull request May 28, 2026
Merged
ml86 added a commit that referenced this pull request May 29, 2026
@ml86
Revert optional parameter index intergration PR (#6011)
6e36fc8 
* Revert "[x2cpg] Fix ParameterIndexCompatPass condition (#6009)"

This reverts commit de0871e.

* Revert "[cpg] QT-676: Migrate parameter index to Option[Int] (#6002)"

This reverts commit 896ce6a.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ml86 ml86 ml86 approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@SuperUserDone @ml86