Skip to content
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

Trait Obligations building block #502

Merged
merged 1 commit into from
Jun 27, 2021
Merged

Trait Obligations building block #502

merged 1 commit into from
Jun 27, 2021

Conversation

philberty
Copy link
Member

@philberty philberty commented Jun 18, 2021

This is a building block for actually enforcing the obligations of a trait.
It includes a query-based lookup of the trait to avoid reloading it so
we only do the hard work once. Then provides the enforcement for
basic functions. Constants, methods and associated types are still
WIP.

Addresses: #440

@philberty philberty self-assigned this Jun 18, 2021
@philberty philberty added this to the Data Structures 3 - Traits milestone Jun 18, 2021
@philberty philberty changed the title trait-obligations wip WIP trait-obligations Jun 18, 2021
@philberty philberty force-pushed the phil/traits-dev-4 branch 3 times, most recently from 2c58aaf to 4104c8a Compare June 25, 2021 14:55
… trait

When we create an impl for a trait each item must be part of the trait,
these are the obligations of the trait. This means we know this type
definetly implements the trait.

This only supports functions so far. Methods, constants and associated
types are still in progress. This is an isolated building block that can
be merged without any regression so far.

Addresses #440
@philberty philberty changed the title WIP trait-obligations Trait Obligations building block Jun 27, 2021
@philberty
Copy link
Member Author

bors r+

@bors
Copy link
Contributor

bors bot commented Jun 27, 2021

Build succeeded:

@bors bors bot merged commit f4668ee into master Jun 27, 2021
@philberty philberty deleted the phil/traits-dev-4 branch June 29, 2021 14:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant