[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-bindings
Subject:    [Kde-bindings]
From:       Arno Rehn <kde () arnorehn ! de>
Date:       2010-01-29 17:33:09
Message-ID: 1264786389.818609.1838.nullmailer () svn ! kde ! org
[Download RAW message or body]

SVN commit 1082090 by arnorehn:

Generate default constructors for classes with pure virtuals, too.

CCMAIL: kde-bindings@kde.org



 M  +2 -2      helpers.cpp  


--- branches/KDE/4.4/kdebindings/generator/generators/smoke/helpers.cpp \
#1082089:1082090 @@ -254,7 +254,7 @@
     // The class can be instanciated if it has a public constructor or no \
constructor at all  // because then it has a default one generated by the compiler.
     // If it has private pure virtuals, then it can't be instanciated either.
-    bool ret = ((publicCtorFound || !ctorFound)/* && !privatePureVirtualsFound*/);
+    bool ret = ((publicCtorFound || !ctorFound) && !privatePureVirtualsFound);
     cache[klass] = ret;
     return ret;
 }
@@ -386,7 +386,7 @@
 {
     foreach (const Method& meth, klass->methods()) {
         // if the class already has a constructor or if it has pure virtuals, \
                there's nothing to do for us
-        if (meth.isConstructor() || meth.flags() & Method::PureVirtual)
+        if (meth.isConstructor())
             return;
         else if (meth.isDestructor() && meth.access() == Access_private)
             return;
_______________________________________________
Kde-bindings mailing list
Kde-bindings@kde.org
https://mail.kde.org/mailman/listinfo/kde-bindings


[prev in list] [next in list] [prev in thread] [next in thread]