Naios / continuable

C++14 asynchronous allocation aware futures (supporting then, exception handling, coroutines and connections)
https://naios.github.io/continuable/
MIT License
815 stars 44 forks source link

Fix incorrect initialization of unsafe_locker #53

Closed rkonklewski-am2m closed 1 year ago

rkonklewski-am2m commented 1 year ago

@Naios

This fixes #51


What was a problem?

List-initialization of unsafe_locker class resulted in a compiler error, because 1) it had no constructor matching the arguments, and 2) it had other user-declared constructors, so aggregate initialization was not allowed (per C++20 rules).

References

How this PR fixes the problem?

This change fixes the issue by adding an appropriate constructor.

Check lists (check x in [ ] of list items)

Additional Comments (if any)

IMO no new unit tests were needed.