Skip to content

Add generalized Typeable #7394

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

nicolasstucki
Copy link
Contributor

@nicolasstucki nicolasstucki commented Oct 9, 2019

See docs/docs/reference/other-new-features/typeable.md

This is one of the possible ways to solve #7204.

Note: This is a generalization of the shapeless Typeable.

Alternatively we could use https://github.com/dotty-staging/dotty/tree/refined-scrutinee

@nicolasstucki nicolasstucki changed the title Add generalized typeable Add generalized Typeable Oct 9, 2019
@nicolasstucki nicolasstucki self-assigned this Oct 9, 2019
It is a generalization of ClassTag with a customizable input type and acting on non erased types.
@nicolasstucki nicolasstucki force-pushed the add-generilized-typable branch from 313efb5 to 72d6dc7 Compare October 30, 2019 07:40
@odersky
Copy link
Contributor

odersky commented Nov 13, 2019

I am a bit wary to add to the zoo of type abstractions. We already have ClassTag, TypeTag, and quoted.Type. What is the overlap with Typeable? Should we not rather fix ClassTag?

@nicolasstucki
Copy link
Contributor Author

I created a minimal example in #7528 of the unsound cases that need to be fixed to be able to use ClassTag.

See the comment in https://github.com/lampepfl/dotty/pull/7528/files#diff-3d1bda775ef3c44c3112f6cc14fdf198

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