Skip to content

Stores, boxed and mapped signals #4413

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

Draft
wants to merge 28 commits into
base: main
Choose a base branch
from

Conversation

ealmloff
Copy link
Member

@ealmloff ealmloff commented Jul 14, 2025

TODO:

  • Make Readable and Writable dyn compatible. Most of the complex logic is split out into extension traits which makes this PR breaking
  • Implement map_mut. This requires two different closures to map the immutable and mutable borrows since closures cannot be generic over mutability
  • Make BoxedReadable Copy
  • Merge ReadOnlySignal and BoxedReadable (and keep ReadOnlySignal as a type alias)

Closes #3610

@ealmloff ealmloff added this to the 0.8.0 milestone Jul 14, 2025
@ealmloff ealmloff added signals Related to the signals crate breaking This is a breaking change labels Jul 14, 2025
@ealmloff ealmloff changed the title Boxed and mapped signals Stores, boxed and mapped signals Jul 16, 2025
@ealmloff ealmloff added the experimental May or may not be merged label Jul 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking This is a breaking change experimental May or may not be merged signals Related to the signals crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Writable::map for Signal Mapping
1 participant