See also: https://codereview.qt-project.org/#/c/216497/ Generally, it should be warned to not use QTimer::singleShot(0, Functor), as that cannot be optimized easily. It should instead use QTimer::singleShot(0, ctx, Functor);
Is this still relevant with all your patches ?
no, once the patches get in we can ignore this. Does clazy know about the used Qt version? If so, it could warn about this in older Qt versions. Low-prio of course
Let's tackle this in Qt instead. And anyway clazy already warns if you don't pass a context object, as that can bring threading bugs or crashes. Also, the general feeling is to deprecate the connect() taking only 3 args, and forcing people to use context obj, even with lambdas/functor.