search

NoAvailableAlias / nano-signal-slot

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

Add emit argument tests #28

Closed NoAvailableAlias closed 4 years ago

NoAvailableAlias commented 4 years ago

Add more emit tests concerning argument copying as well as forwarding issues. For reference:

https://github.com/j-jorge/signal-slot-benchmarks/commit/ccfe06fadc3852ca99b0bb4ec75d48d264489447

Resolve any issues found in nano-signal-slot concerning the results.

NoAvailableAlias commented 4 years ago

Better explanation of the problem:

https://www.reddit.com/r/cpp/comments/gq5cbt/testing_c_signalslot_libraries/frsi00p?utm_source=share&utm_medium=web2x

Changes have been completed and are now being tested.