AuroraMiddleware/qt5base-lts
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

moc: fix Q_INVOKABLE returning references

Browse Source 
The moc generated code would not compile otherwise

Keep Moc::parseFunction and Moc::parseMaybeFunction in sync
(the first is used for signals and slots, and the second for normal
 functions such as Q_INVOKABLE)
Last patch that introduced function pointer updated parseFunction
but not parseMaybeFunction

When a slot return a reference, moc generate code that make the
MetaObject system think it is a void, so qt_metacall and invokeMethod
do not mess with the return value.
But when we want to take the function signature, in the IndexOfMethod
call, we need to have the exact return type.

Change-Id: I4661218d7ce367ad3934e73929e7d04f0a6dbc09
Reviewed-by: Bradley T. Hughes <bradley.hughes@nokia.com>
This commit is contained in:
Olivier Goffart 2011-11-15 10:14:23 +01:00 committed by Qt by Nokia
parent a254611ba1
commit 9924a637cc
2 changed files with 20 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/tools/moc/moc.cpp
View File

@ -503,8 +503,11 @@ bool Moc::parseMaybeFunction(const ClassDef *cdef, FunctionDef *def)
} }
// we don't support references as return types, it's too dangerous // we don't support references as return types, it's too dangerous
if (def->type.referenceType == Type::Reference) if (def->type.referenceType == Type::Reference) {
QByteArray rawName = def->type.rawName;
def->type = Type("void"); def->type = Type("void");
def->type.rawName = rawName;
}
def->normalizedType = normalizeType(def->type.name); def->normalizedType = normalizeType(def->type.name);

16
tests/auto/tools/moc/tst_moc.cpp
View File

@ -373,6 +373,11 @@ public slots:
public: public:
Q_INVOKABLE void const slotWithSillyConst2() {} Q_INVOKABLE void const slotWithSillyConst2() {}
Q_INVOKABLE QObject& myInvokableReturningRef()
{ return *this; }
Q_INVOKABLE const QObject& myInvokableReturningConstRef() const
{ return *this; }
// that one however should be fine // that one however should be fine
public slots: public slots:
@ -530,6 +535,7 @@ private slots:
void privateClass(); void privateClass();
void cxx11Enums_data(); void cxx11Enums_data();
void cxx11Enums(); void cxx11Enums();
void returnRefs();
signals: signals:
void sigWithUnsignedArg(unsigned foo); void sigWithUnsignedArg(unsigned foo);
@ -1687,6 +1693,16 @@ void tst_Moc::cxx11Enums()
} }
} }
void tst_Moc::returnRefs()
{
TestClass tst;
const QMetaObject *mobj = tst.metaObject();
QVERIFY(mobj->indexOfMethod("myInvokableReturningRef()") != -1);
QVERIFY(mobj->indexOfMethod("myInvokableReturningConstRef()") != -1);
// Those two functions are copied from the qscriptextqobject test in qtscript
// they used to cause miscompilation of the moc generated file.
}
QTEST_APPLESS_MAIN(tst_Moc) QTEST_APPLESS_MAIN(tst_Moc)
#include "tst_moc.moc" #include "tst_moc.moc"