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

Remove payload from HandleUpdate? #308

Open
dhardy opened this issue Apr 25, 2022 · 1 comment
Open

Remove payload from HandleUpdate? #308

dhardy opened this issue Apr 25, 2022 · 1 comment
Labels
discussion Requires analysis/discussion

Comments

@dhardy
Copy link
Collaborator

dhardy commented Apr 25, 2022

It was at one point used to implement RadioBox but now appears to have no use. It is potentially useful, e.g. examples/async-event could encode its hue and pass via this instead of using an Arc<Mutex>. Some options:

  1. Remove
  2. Keep as is
  3. Replace with Option<Box<dyn Any>>?
@dhardy dhardy added the discussion Requires analysis/discussion label Apr 25, 2022
@dhardy
Copy link
Collaborator Author

dhardy commented Jul 16, 2022

This is now Event::Update, which still supports a payload: u64, but is never used.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion Requires analysis/discussion
Projects
None yet
Development

No branches or pull requests

1 participant