Skip to content

Remove runtime.IfaceIndir#563

Merged
goccy merged 2 commits intonew-enginefrom
ifaceindir
Jan 4, 2026
Merged

Remove runtime.IfaceIndir#563
goccy merged 2 commits intonew-enginefrom
ifaceindir

Conversation

@goccy
Copy link
Owner

@goccy goccy commented Jan 4, 2026

To eliminate the use of linkname, we first remove the dependency on ifaceIndir, which is an internal API of the reflect package. Along with this, we've improved the processing in the encoder's compiler.

@codecov-commenter
Copy link

codecov-commenter commented Jan 4, 2026

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

❌ Patch coverage is 83.93782% with 31 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.67%. Comparing base (e4877d5) to head (32457a5).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@              Coverage Diff               @@
##           new-engine     #563      +/-   ##
==============================================
+ Coverage       78.50%   78.67%   +0.17%     
==============================================
  Files              55       55              
  Lines           16717    16721       +4     
==============================================
+ Hits            13124    13156      +32     
+ Misses           2960     2924      -36     
- Partials          633      641       +8     
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@goccy goccy merged commit a72016c into new-engine Jan 4, 2026
13 checks passed
@goccy goccy deleted the ifaceindir branch January 4, 2026 04:54
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