Skip to content

Fix name overrides for constructed types#552

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

Fix name overrides for constructed types#552
ds5678 wants to merge 1 commit into
SamboyCoding:developmentfrom
AssetRipper:fix-name-overrides

Conversation

@ds5678
Copy link
Copy Markdown
Contributor

@ds5678 ds5678 commented May 17, 2026

  • Ensure that DefaultName only calls DefaultName and DefaultFullName, not Name or FullName
  • Remove unnecessary ReferenceTypeAnalysisContext::ToString override
  • Seal Type and name properties
  • Use DefaultFullName and FullName for arguments of GenericInstanceTypeAnalysisContext to ensure expected output from its own DefaultFullName and FullName

* Ensure that DefaultName only calls DefaultName and DefaultFullName, not Name or FullName
* Remove unnecessary ReferenceTypeAnalysisContext::ToString override
* Seal Type and name properties
* Use DefaultFullName and FullName for arguments of GenericInstanceTypeAnalysisContext to ensure expected output from its own DefaultFullName and FullName
@coveralls
Copy link
Copy Markdown

Coverage Report for CI Build 26005967564

Coverage decreased (-0.06%) to 35.273%

Details

  • Coverage decreased (-0.06%) from the base build.
  • Patch coverage: 51 uncovered changes across 11 files (2 of 53 lines covered, 3.77%).
  • 1 coverage regression across 1 file.

Uncovered Changes

Top 10 Files by Coverage Impact Changed Covered %
Cpp2IL.Core/Model/Contexts/GenericInstanceTypeAnalysisContext.cs 9 0 0.0%
Cpp2IL.Core/Model/Contexts/CustomModifierTypeAnalysisContext.cs 8 0 0.0%
Cpp2IL.Core/Model/Contexts/SentinelTypeAnalysisContext.cs 5 0 0.0%
Cpp2IL.Core/Model/Contexts/ArrayTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/BoxedTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/ByRefTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/PinnedTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/PointerTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/SzArrayTypeAnalysisContext.cs 4 0 0.0%
Cpp2IL.Core/Model/Contexts/WrappedTypeAnalysisContext.cs 3 0 0.0%

Coverage Regressions

1 previously-covered line in 1 file lost coverage.

File Lines Losing Coverage Coverage
Cpp2IL.Core/Model/Contexts/SentinelTypeAnalysisContext.cs 1 0.0%

Coverage Stats

Coverage Status
Relevant Lines: 11365
Covered Lines: 4495
Line Coverage: 39.55%
Relevant Branches: 6975
Covered Branches: 1974
Branch Coverage: 28.3%
Branches in Coverage %: Yes
Coverage Strength: 252024.2 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