search

Pauan / rust-dominator

Zero-cost ultra-high-performance declarative DOM library using FRP signals for Rust!
MIT License
999 stars 63 forks source link

Add .broadcast() #5

Closed davidhewitt closed 6 years ago

davidhewitt commented 6 years ago

I had a go at adding the .broadcast() method as discussed in #3 . Hopefully you like it! 😄

davidhewitt commented 6 years ago

Thanks for the rapid review! I'll have a read of futures::shared and take another shot at this later in the week.

Pauan commented 6 years ago

@davidhewitt Here is the source code for the Future::shared method.

I think we'll need to do something similar. The use of Arc is unfortunate for performance, but probably necessary.

davidhewitt commented 6 years ago

Closing; will re-open new PR in rust-signals.