Fix bug 616 - "error compiling when public and private methods differ by the const-ness"

Reviewer: Marcelo Lira <marcelo.lira@openbossa.org>
          Renato Araújo <renato.filho@openbossa.org>

3 files changed, 57 insertions, 7 deletions

diff --git a/generator/cppgenerator.cpp b/generator/cppgenerator.cpp
index d5697ab72..b7148017c 100644
--- a/generator/cppgenerator.cpp
+++ b/generator/cppgenerator.cpp
@@ -2092,12 +2092,18 @@ void CppGenerator::writeMethodCall(QTextStream& s, const AbstractMetaFunction* f
                         if (func->isStatic())
                             mc << func->ownerClass()->qualifiedCppName() << "::";
                         else {
+                            if (func->isConstant()) {
 #ifdef AVOID_PROTECTED_HACK
-                            if (!func->isVirtual() && func->ownerClass()->hasProtectedMembers())
-                                mc << "((" << func->ownerClass()->qualifiedCppName() << "*)" << CPP_SELF_VAR << ")->";
-                            else
+                                mc << "const_cast<const ";
+                                bool hasProtectedMembers = func->ownerClass()->hasProtectedMembers();
+                                mc << (hasProtectedMembers ? wrapperName(func->ownerClass()) : func->ownerClass()->qualifiedCppName());
+                                mc <<  "*>(" CPP_SELF_VAR ")->";
+#else
+                                mc << "const_cast<const " << func->ownerClass()->qualifiedCppName() <<  "*>(" CPP_SELF_VAR ")->";
 #endif
+                            } else {
                                 mc << CPP_SELF_VAR "->";
+                            }
                         }
 
                         if (!func->isAbstract() && func->isVirtual())
diff --git a/generator/headergenerator.cpp b/generator/headergenerator.cpp
index bb8f341a1..b7e2bbdc9 100644
--- a/generator/headergenerator.cpp
+++ b/generator/headergenerator.cpp
@@ -73,6 +73,7 @@ void HeaderGenerator::writeProtectedFieldAccessors(QTextStream& s, const Abstrac
 void HeaderGenerator::generateClass(QTextStream& s, const AbstractMetaClass* metaClass)
 {
     ReportHandler::debugSparse("Generating header for " + metaClass->fullName());
+    m_inheritedOverloads.clear();
     Indentation indent(INDENT);
 
     // write license comment
@@ -143,13 +144,18 @@ void HeaderGenerator::generateClass(QTextStream& s, const AbstractMetaClass* met
         }
 #endif
 
+        if (m_inheritedOverloads.size()) {
+            s << INDENT << "// Inherited overloads, because the using keyword sux" << endl;
+            writeInheritedOverloads(s);
+        }
+
         s << "};" << endl << endl;
     }
 
     s << "#endif // SBK_" << headerGuard << "_H" << endl << endl;
 }
 
-void HeaderGenerator::writeFunction(QTextStream& s, const AbstractMetaFunction* func) const
+void HeaderGenerator::writeFunction(QTextStream& s, const AbstractMetaFunction* func)
 {
 
     // do not write copy ctors here.
@@ -206,6 +212,20 @@ void HeaderGenerator::writeFunction(QTextStream& s, const AbstractMetaFunction*
 
         s << functionSignature(func, "", "", virtualOption) << ';' << endl;
 
+        // Check if this method hide other methods in base classes
+        foreach (const AbstractMetaFunction* f, func->ownerClass()->functions()) {
+            if (f != func
+                && !f->isConstructor()
+                && !f->isPrivate()
+                && !f->isVirtual()
+                && !f->isAbstract()
+                && !f->isStatic()
+                && f->name() == func->name()) {
+                m_inheritedOverloads << f;
+                break;
+            }
+        }
+
         // TODO: when modified an abstract method ceases to be virtual but stays abstract
         //if (func->isModifiedRemoved() && func->isAbstract()) {
         //}
@@ -621,3 +641,27 @@ void HeaderGenerator::writeTypeConverterImpl(QTextStream& s, const TypeEntry* ty
     s << '}' << endl << endl;
 }
 
+void HeaderGenerator::writeInheritedOverloads(QTextStream& s)
+{
+    foreach (const AbstractMetaFunction* func, m_inheritedOverloads) {
+        s << INDENT << "inline ";
+        s << functionSignature(func, "", "", Generator::EnumAsInts|Generator::OriginalTypeDescription) << " { ";
+        s << (func->type() ? "return " : "");
+        s << func->ownerClass()->qualifiedCppName() << "::" << func->originalName() << '(';
+        QStringList args;
+        foreach (const AbstractMetaArgument* arg, func->arguments()) {
+            QString argName = arg->name();
+            const TypeEntry* enumTypeEntry = 0;
+            if (arg->type()->isFlags())
+                enumTypeEntry = reinterpret_cast<const FlagsTypeEntry*>(arg->type()->typeEntry())->originator();
+            else if (arg->type()->isEnum())
+                enumTypeEntry = arg->type()->typeEntry();
+            if (enumTypeEntry)
+                argName = QString("%1(%2)").arg(arg->type()->cppSignature()).arg(argName);
+            args << argName;
+        }
+        s << args.join(", ") << ')';
+        s << "; }" << endl;
+    }
+}
+
diff --git a/generator/headergenerator.h b/generator/headergenerator.h
index d7502a9e1..6710b9948 100644
--- a/generator/headergenerator.h
+++ b/generator/headergenerator.h
@@ -41,9 +41,7 @@ protected:
 private:
     void writeCopyCtor(QTextStream &s, const AbstractMetaClass* metaClass) const;
     void writeProtectedFieldAccessors(QTextStream& s, const AbstractMetaField* field) const;
-    void writeFunction(QTextStream& s, const AbstractMetaFunction* func) const;
-    void writePureVirtualEmptyImpl(QTextStream& , const AbstractMetaFunction* func) const;
-    void writeDefaultImplementation(QTextStream& s, const AbstractMetaFunction* func) const;
+    void writeFunction(QTextStream& s, const AbstractMetaFunction* func);
     void writeTypeConverterDecl(QTextStream& s, const TypeEntry* type);
     void writeSbkTypeFunction(QTextStream& s, const AbstractMetaEnum* cppEnum);
     void writeSbkTypeFunction(QTextStream& s, const AbstractMetaClass* cppClass);
@@ -51,7 +49,9 @@ private:
     void writeTypeIndexDefine(QTextStream& s, const AbstractMetaClass* metaClass, int& idx);
     void writeTypeConverterImpl(QTextStream& s, const TypeEntry* type);
     void writeProtectedEnumSurrogate(QTextStream& s, const AbstractMetaEnum* cppEnum);
+    void writeInheritedOverloads(QTextStream& s);
 
+    QSet<const AbstractMetaFunction*> m_inheritedOverloads;
 };
 
 #endif // HEADERGENERATOR_H