Skip to content

Fix some build warnings#562

Merged
SamboyCoding merged 1 commit into
SamboyCoding:developmentfrom
AssetRipper:fix-warnings
Jun 20, 2026
Merged

Fix some build warnings#562
SamboyCoding merged 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

@SamboyCoding SamboyCoding merged commit 5d99a94 into SamboyCoding:development Jun 20, 2026
9 checks passed
Repository owner locked as resolved and limited conversation to collaborators Jun 20, 2026
@ds5678 ds5678 deleted the fix-warnings branch June 20, 2026 16:47
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants