danvratil / qcoro

C++ Coroutines for Qt
https://qcoro.dvratil.cz
MIT License
326 stars 53 forks source link

QCoro::connect: Fix bug in concept requirements #179

Closed jbruechert closed 1 year ago

jbruechert commented 1 year ago

We are currently testing whether the lambda is invokable with the task as argument, which it should not be.

Weirdly enough this actually works with modern versions of clang and gcc, but I'm pretty sure its still wrong.