sashakoshka/hopp
1
0
Fork 0
Code Issues 8 Pull Requests Actions Packages Projects Releases Wiki Activity

tape.TagAny should be using reflection #10

New Issue
Closed
opened 2025-08-10 21:51:25 -06:00 by sashakoshka · 0 comments
sashakoshka commented 2025-08-10 21:51:25 -06:00
Owner
Copy Link

Using a type switch is unacceptable because we need to match underlying types.

We should have a private tagAny that takes in a reflect.Value, and a public TagAny with the current interface. That way, the API can be normal, but the internal functions can directly pass the reflect.Value without it having to be re-created.

Using a type switch is unacceptable because we need to match underlying types. We should have a private `tagAny` that takes in a reflect.Value, and a public `TagAny` with the current interface. That way, the API can be normal, but the internal functions can directly pass the reflect.Value without it having to be re-created.
Sign in to join this conversation.
No Branch/Tag Specified
Branches Tags
Labels
No items
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
sashakoshka
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: sashakoshka/hopp#10
No description provided.
Delete Branch "%!s()"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?