Skip to content

Conversation

ds5678
Copy link
Contributor

@ds5678 ds5678 commented May 13, 2025

Modified the IsRef property to check for ByRefTypeAnalysisContext instead of ReferencedTypeAnalysisContext, correcting the behavior of reference parameter handling.

@coveralls
Copy link

coveralls commented May 13, 2025

Pull Request Test Coverage Report for Build 15043797356

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

Details

  • 0 of 1 (0.0%) changed or added relevant line in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage increased (+0.002%) to 34.797%

Changes Missing Coverage Covered Lines Changed/Added Lines %
Cpp2IL.Core/Model/Contexts/ParameterAnalysisContext.cs 0 1 0.0%
Totals Coverage Status
Change from base Build 15043776015: 0.002%
Covered Lines: 4159
Relevant Lines: 10653

💛 - Coveralls

ds5678 added 2 commits May 15, 2025 12:28
Modified the IsRef property to check for ByRefTypeAnalysisContext instead of ReferencedTypeAnalysisContext, changing the behavior of reference parameter handling.
@SamboyCoding SamboyCoding force-pushed the fix-inject-parameter-isref branch from 7eead67 to 9133368 Compare May 15, 2025 11:28
@SamboyCoding SamboyCoding merged commit 05ce732 into SamboyCoding:development May 15, 2025
5 checks passed
@ds5678 ds5678 deleted the fix-inject-parameter-isref branch May 15, 2025 17:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants