Skip to content

Fix some build warnings#562

Open
ds5678 wants to merge 1 commit into
SamboyCoding:developmentfrom
AssetRipper:fix-warnings
Open

Fix some build warnings#562
ds5678 wants to merge 1 commit into
SamboyCoding:developmentfrom
AssetRipper:fix-warnings

Conversation

@ds5678

@ds5678 ds5678 commented Jun 16, 2026

Copy link
Copy Markdown
Contributor

Mostly nullable warnings

@coveralls

Copy link
Copy Markdown

Coverage Report for CI Build 27655478671

Coverage remained the same at 35.351%

Details

  • Coverage remained the same as the base build.
  • Patch coverage: 2 uncovered changes across 2 files (2 of 4 lines covered, 50.0%).
  • 1 coverage regression across 1 file.

Uncovered Changes

File Changed Covered %
Cpp2IL.Core/Model/Contexts/ApplicationAnalysisContext.cs 1 0 0.0%
Cpp2IL.Core/Model/Contexts/TypeAnalysisContext.cs 1 0 0.0%
Total (4 files) 4 2 50.0%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
Cpp2IL.Core/Model/Contexts/ApplicationAnalysisContext.cs 1 62.82%

Coverage Stats

Coverage Status
Relevant Lines: 12583
Covered Lines: 4851
Line Coverage: 38.55%
Relevant Branches: 8050
Covered Branches: 2443
Branch Coverage: 30.35%
Branches in Coverage %: Yes
Coverage Strength: 238886.78 hits per line

💛 - Coveralls

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.

2 participants