From 4570d0ac2401f3902cff1bd72cac78f599103096 Mon Sep 17 00:00:00 2001 From: Marc Mutz Date: Mon, 20 Aug 2012 15:17:54 +0200 Subject: QSlotObjectBase: remove misleading comment These constructors might not benefit from being constexpr (as the objects are only created on the heap), so don't suggest so. There's no disadvantage of them begin constexpr, but their constexpr'ability depends on whether QAtomic has a constexpr constructor, and the added complexity of finding that out isn't worth it. Change-Id: I089a29dcb98ba935c339dce09d71f283522a9afd Reviewed-by: Thiago Macieira Reviewed-by: Olivier Goffart --- src/corelib/kernel/qobject_impl.h | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/corelib') diff --git a/src/corelib/kernel/qobject_impl.h b/src/corelib/kernel/qobject_impl.h index f5187c803a..3586b1c7f7 100644 --- a/src/corelib/kernel/qobject_impl.h +++ b/src/corelib/kernel/qobject_impl.h @@ -116,7 +116,7 @@ namespace QtPrivate { typedef void (*ImplFn)(int which, QSlotObjectBase* this_, QObject *receiver, void **args, bool *ret); const ImplFn impl; - explicit QSlotObjectBase(ImplFn fn) : ref(1), impl(fn) {} // ### make constexpr once QAtomicInt's ctor is, too + explicit QSlotObjectBase(ImplFn fn) : ref(1), impl(fn) {} inline void destroy() { impl(Destroy, this, 0, 0, 0); } inline bool compare(void **a) { bool ret; impl(Compare, this, 0, a, &ret); return ret; } inline void call(QObject *r, void **a) { impl(Call, this, r, a, 0); } -- cgit v1.2.3