From 22494ea8e82d5d5588e67e8067f0273687ed9098 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?J=C4=99drzej=20Nowacki?= <jedrzej.nowacki@nokia.com>
Date: Tue, 13 Mar 2012 14:15:53 +0100
Subject: Unify QMetaType::TypeFlags detection.

Duplicated code was removed. As an side effect:
- one runtime flag check was replaced by a compile time check.
- is enum flag can be used together with built-in types.

Change-Id: I54173e7b07ce7e487d3cc21ba24dcccd28b5d049
Reviewed-by: Olivier Goffart <ogoffart@woboq.com>
---
 src/corelib/kernel/qmetatype.h | 24 ++++++++++++------------
 1 file changed, 12 insertions(+), 12 deletions(-)

(limited to 'src/corelib/kernel/qmetatype.h')

diff --git a/src/corelib/kernel/qmetatype.h b/src/corelib/kernel/qmetatype.h
index 0010c27beb..bf1b126200 100644
--- a/src/corelib/kernel/qmetatype.h
+++ b/src/corelib/kernel/qmetatype.h
@@ -435,6 +435,17 @@ namespace QtPrivate {
     template <class Result, class Arg0> struct IsPointerToTypeDerivedFromQObject<Result(*)(Arg0)> { enum { Value = false }; };
     template <class Result, class Arg0, class Arg1> struct IsPointerToTypeDerivedFromQObject<Result(*)(Arg0, Arg1)> { enum { Value = false }; };
     template <class Result, class Arg0, class Arg1, class Arg2> struct IsPointerToTypeDerivedFromQObject<Result(*)(Arg0, Arg1, Arg2)> { enum { Value = false }; };
+
+    template<typename T>
+    struct QMetaTypeTypeFlags
+    {
+        enum { Flags = (!QTypeInfo<T>::isStatic ? QMetaType::MovableType : 0)
+                     | (QTypeInfo<T>::isComplex ? QMetaType::NeedsConstruction : 0)
+                     | (QTypeInfo<T>::isComplex ? QMetaType::NeedsDestruction : 0)
+                     | (IsPointerToTypeDerivedFromQObject<T>::Value ? QMetaType::PointerToQObject : 0)
+                     | (Q_IS_ENUM(T) ? QMetaType::IsEnumeration : 0)
+             };
+    };
 }
 
 template <typename T>
@@ -448,18 +459,7 @@ int qRegisterMetaType(const char *typeName
     if (typedefOf != -1)
         return QMetaType::registerTypedef(typeName, typedefOf);
 
-    QMetaType::TypeFlags flags;
-    if (!QTypeInfo<T>::isStatic)
-        flags |= QMetaType::MovableType;
-    if (QTypeInfo<T>::isComplex) {
-        flags |= QMetaType::NeedsConstruction;
-        flags |= QMetaType::NeedsDestruction;
-    }
-    if (QtPrivate::IsPointerToTypeDerivedFromQObject<T>::Value)
-        flags |= QMetaType::PointerToQObject;
-    if (Q_IS_ENUM(T))
-        flags |= QMetaType::IsEnumeration;
-
+    QMetaType::TypeFlags flags(QtPrivate::QMetaTypeTypeFlags<T>::Flags);
     return QMetaType::registerType(typeName, qMetaTypeDeleteHelper<T>,
                                    qMetaTypeCreateHelper<T>,
                                    qMetaTypeDestructHelper<T>,
-- 
cgit v1.2.3