Skip to content

Diffable CS improvements#577

Open
ds5678 wants to merge 5 commits into
SamboyCoding:developmentfrom
AssetRipper:diffable-cs-improvements
Open

Diffable CS improvements#577
ds5678 wants to merge 5 commits into
SamboyCoding:developmentfrom
AssetRipper:diffable-cs-improvements

Conversation

@ds5678

@ds5678 ds5678 commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

Improvements:

  • Handle all access modifiers
  • Support nint and nuint
  • Added properties to EventAnalysisContext, PropertyAnalysisContext, MethodAnalysisContext, and TypeAnalysisContext
  • Support delegate type kind
  • Improve accuracy of the virtual lookup keywords
  • Switch from StringBuilder to IndentedTextWriter (AI-written)

@coveralls

coveralls commented Jul 7, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 28842709595

Coverage increased (+0.2%) to 36.304%

Details

  • Coverage increased (+0.2%) from the base build.
  • Patch coverage: 18 uncovered changes across 4 files (249 of 267 lines covered, 93.26%).
  • No coverage regressions found.

Uncovered Changes

File Changed Covered %
Cpp2IL.Core/Utils/CsFileUtils.cs 107 97 90.65%
Cpp2IL.Core/OutputFormats/DiffableCsOutputFormat.cs 123 119 96.75%
Cpp2IL.Core/Model/Contexts/EventAnalysisContext.cs 18 15 83.33%
Cpp2IL.Core/Model/Contexts/PropertyAnalysisContext.cs 17 16 94.12%
Total (6 files) 267 249 93.26%

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 12948
Covered Lines: 5078
Line Coverage: 39.22%
Relevant Branches: 8570
Covered Branches: 2734
Branch Coverage: 31.9%
Branches in Coverage %: Yes
Coverage Strength: 170756.73 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