Skip to content

Fix incorrect handling of MetatypeType (T.Type) - #333

Merged
sidepelican merged 1 commit into
uber:masterfrom
sidepelican:metatypetype_fix
Feb 8, 2026
Merged

sidepelican merged 1 commit into
uber:masterfrom
sidepelican:metatypetype_fix

Conversation

@sidepelican

Copy link
Copy Markdown
Collaborator

I noticed that metatypeType was not being processed correctly.
Since T.Type is a generic metatype, it cannot be directly referenced as a concrete type for a handler in this context. It should have been treated as Any.

  • target code
func useMetatype<T>(type: T.Type) -> T.Type
  • before
var useMetatypeHandler: ((T.Type) -> T.Type)?
...
  • after
var useMetatypeHandler: ((Any) -> Any)?
...

@sidepelican
sidepelican requested a review from fummicc1 February 5, 2026 04:06

@fummicc1 fummicc1 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@sidepelican
sidepelican merged commit fff2155 into uber:master Feb 8, 2026
8 checks passed
@sidepelican
sidepelican deleted the metatypetype_fix branch February 8, 2026 12:50
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