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.

Sign up for GitHub

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

Jump to bottom

Large update and re-work of the value type system #43

Merged
merged 1 commit into from
Oct 26, 2023
Merged

Large update and re-work of the value type system #43

merged 1 commit into from
Oct 26, 2023

Conversation

Licenser
Copy link
Member

@Licenser Licenser commented Oct 25, 2023 •
edited
Loading

This update comes with multiple changes big:

  1. the traits are now finer grained, isolating container types from scalar types and mutable functions from purely accessible functions
  2. the traits no longer depend on each other and remove all default implementations that were based on type dependency
  3. there are now blanked implementations providing automatic implementations of advanced traits if a combination of simpler traits are implemented
  4. type bounds are significantly reduced and relaxed to allow a wider set of types to implement the traits

mfelsche
mfelsche reviewed Oct 25, 2023
View reviewed changes
src/base.rs Outdated Show resolved Hide resolved
@Licenser Licenser force-pushed the trait-split branch 2 times, most recently from 6ffea39 to 3cf3fe1 Compare October 25, 2023 13:22
@Licenser Licenser marked this pull request as ready for review October 25, 2023 13:23
@Licenser Licenser changed the title WIP Large update and re-work of the value type system Oct 25, 2023
@Licenser Licenser requested a review from mfelsche October 25, 2023 13:29
@Licenser
Update of traits
aa81a1e 
- traits are now finer grained
- traints no longer depend on each other
- traits come with default implementations for traits that can be
  derived

Signed-off-by: Heinz N. Gies <heinz@licenser.net>
@Licenser Licenser mentioned this pull request Oct 25, 2023
Closed
@Licenser Licenser merged commit fb54c8e into main Oct 26, 2023
@Licenser Licenser deleted the trait-split branch October 26, 2023 06:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mfelsche mfelsche Awaiting requested review from mfelsche

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Licenser @mfelsche