NoAvailableAlias / nano-signal-slot

Pure C++17 Signals and Slots
MIT License
407 stars 60 forks source link

Issue with Copying #15

Closed NoAvailableAlias closed 4 years ago

NoAvailableAlias commented 8 years ago

Nano-signal-slot currently does not like to be copied. Before the recent merge from FT this was ok due to using the std::map structure. For now Nano::Observer will need to be made noncopyable by adding deleted constructors. The ~first~ nano-signal-slot release will resolve this issue.

Edit: The first release was meant as a tag. Not sure if resolving this or not.

NoAvailableAlias commented 4 years ago

Copying is extremely difficult to support in 4 different threading policies. The usefulness of adding copy support does not outweigh the added complexity and dimension of testing and support!

Copying is RIP