diff options
author | Kevin Ottens <kevin.ottens@kdab.com> | 2012-11-30 11:51:05 +0100 |
---|---|---|
committer | The Qt Project <gerrit-noreply@qt-project.org> | 2012-12-11 07:41:49 +0100 |
commit | 5b3e7c8dfa4856b04013ace24d19bbd104dc3bd0 (patch) | |
tree | dbaf1fa3c3ecf623b99807bc18c90e57de535991 /src/dbus | |
parent | fac02ddc39145963646ad0f36b9e41573e49e6c5 (diff) |
Generate the introspection string during parsing
Commit 0696071316b3dacb8d1ca15a269e4f4215642b9d moved away from QDom but
also lost the ability to fill the introspection field of the created
QDBusIntrospection::Interface instances. This commit now generate the
string again as we proceed with the QXmlStreamReader based parsing.
Task-number: QTBUG-26668
Change-Id: I8f406e1f4e9d3e667a8557db69da36cac369ba4f
Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
Diffstat (limited to 'src/dbus')
-rw-r--r-- | src/dbus/qdbusxmlparser.cpp | 156 |
1 files changed, 101 insertions, 55 deletions
diff --git a/src/dbus/qdbusxmlparser.cpp b/src/dbus/qdbusxmlparser.cpp index 315b4a2e24..c2c1b8daff 100644 --- a/src/dbus/qdbusxmlparser.cpp +++ b/src/dbus/qdbusxmlparser.cpp @@ -59,7 +59,8 @@ QT_BEGIN_NAMESPACE -static bool parseArg(const QXmlStreamAttributes &attributes, QDBusIntrospection::Argument &argData) +static bool parseArg(const QXmlStreamAttributes &attributes, QDBusIntrospection::Argument &argData, + QDBusIntrospection::Interface *ifaceData) { const QString argType = attributes.value(QLatin1String("type")).toString(); @@ -72,10 +73,21 @@ static bool parseArg(const QXmlStreamAttributes &attributes, QDBusIntrospection: argData.name = attributes.value(QLatin1String("name")).toString(); argData.type = argType; + ifaceData->introspection += QLatin1String(" <arg"); + if (attributes.hasAttribute(QLatin1String("direction"))) { + const QString direction = attributes.value(QLatin1String("direction")).toString(); + ifaceData->introspection += QLatin1String(" direction=\"") + direction + QLatin1String("\""); + } + ifaceData->introspection += QLatin1String(" type=\"") + argData.type + QLatin1String("\""); + if (!argData.name.isEmpty()) + ifaceData->introspection += QLatin1String(" name=\"") + argData.name + QLatin1String("\""); + ifaceData->introspection += QLatin1String("/>\n"); + return ok; } -static bool parseAnnotation(const QXmlStreamReader &xml, QDBusIntrospection::Annotations &annotations) +static bool parseAnnotation(const QXmlStreamReader &xml, QDBusIntrospection::Annotations &annotations, + QDBusIntrospection::Interface *ifaceData, bool interfaceAnnotation = false) { Q_ASSERT(xml.isStartElement() && xml.name() == QLatin1String("annotation")); @@ -87,12 +99,16 @@ static bool parseAnnotation(const QXmlStreamReader &xml, QDBusIntrospection::Ann qPrintable(name)); return false; } - annotations.insert(name, attributes.value(QLatin1String("value")).toString()); + const QString value = attributes.value(QLatin1String("value")).toString(); + annotations.insert(name, value); + if (!interfaceAnnotation) + ifaceData->introspection += QLatin1String(" "); + ifaceData->introspection += QLatin1String(" <annotation value=\"") + value + QLatin1String("\" name=\"") + name + QLatin1String("\"/>\n"); return true; } static bool parseProperty(QXmlStreamReader &xml, QDBusIntrospection::Property &propertyData, - const QString &ifaceName) + QDBusIntrospection::Interface *ifaceData) { Q_ASSERT(xml.isStartElement() && xml.name() == QLatin1String("property")); @@ -100,7 +116,7 @@ static bool parseProperty(QXmlStreamReader &xml, QDBusIntrospection::Property &p const QString propertyName = attributes.value(QLatin1String("name")).toString(); if (!QDBusUtil::isValidMemberName(propertyName)) { qDBusParserError("Invalid D-BUS member name '%s' found in interface '%s' while parsing introspection", - qPrintable(propertyName), qPrintable(ifaceName)); + qPrintable(propertyName), qPrintable(ifaceData->name)); xml.skipCurrentElement(); return false; } @@ -112,7 +128,7 @@ static bool parseProperty(QXmlStreamReader &xml, QDBusIntrospection::Property &p if (!QDBusUtil::isValidSingleSignature(propertyData.type)) { // cannot be! qDBusParserError("Invalid D-BUS type signature '%s' found in property '%s.%s' while parsing introspection", - qPrintable(propertyData.type), qPrintable(ifaceName), + qPrintable(propertyData.type), qPrintable(ifaceData->name), qPrintable(propertyName)); } @@ -125,18 +141,28 @@ static bool parseProperty(QXmlStreamReader &xml, QDBusIntrospection::Property &p propertyData.access = QDBusIntrospection::Property::ReadWrite; else { qDBusParserError("Invalid D-BUS property access '%s' found in property '%s.%s' while parsing introspection", - qPrintable(access), qPrintable(ifaceName), + qPrintable(access), qPrintable(ifaceData->name), qPrintable(propertyName)); return false; // invalid one! } - while (xml.readNextStartElement()) { - if (xml.name() == QLatin1String("annotation")) { - parseAnnotation(xml, propertyData.annotations); - } else if (xml.prefix().isEmpty()) { - qDBusParserError() << "Unknown element" << xml.name() << "while checking for annotations"; - } - xml.skipCurrentElement(); + ifaceData->introspection += QLatin1String(" <property access=\"") + access + QLatin1String("\" type=\"") + propertyData.type + QLatin1String("\" name=\"") + propertyName + QLatin1String("\""); + + if (!xml.readNextStartElement()) { + ifaceData->introspection += QLatin1String("/>\n"); + } else { + ifaceData->introspection += QLatin1String(">\n"); + + do { + if (xml.name() == QLatin1String("annotation")) { + parseAnnotation(xml, propertyData.annotations, ifaceData); + } else if (xml.prefix().isEmpty()) { + qDBusParserError() << "Unknown element" << xml.name() << "while checking for annotations"; + } + xml.skipCurrentElement(); + } while (xml.readNextStartElement()); + + ifaceData->introspection += QLatin1String(" </property>\n"); } if (!xml.isEndElement() || xml.name() != QLatin1String("property")) { @@ -148,7 +174,7 @@ static bool parseProperty(QXmlStreamReader &xml, QDBusIntrospection::Property &p } static bool parseMethod(QXmlStreamReader &xml, QDBusIntrospection::Method &methodData, - const QString &ifaceName) + QDBusIntrospection::Interface *ifaceData) { Q_ASSERT(xml.isStartElement() && xml.name() == QLatin1String("method")); @@ -156,35 +182,44 @@ static bool parseMethod(QXmlStreamReader &xml, QDBusIntrospection::Method &metho const QString methodName = attributes.value(QLatin1String("name")).toString(); if (!QDBusUtil::isValidMemberName(methodName)) { qDBusParserError("Invalid D-BUS member name '%s' found in interface '%s' while parsing introspection", - qPrintable(methodName), qPrintable(ifaceName)); + qPrintable(methodName), qPrintable(ifaceData->name)); return false; } methodData.name = methodName; + ifaceData->introspection += QLatin1String(" <method name=\"") + methodName + QLatin1String("\""); QDBusIntrospection::Arguments outArguments; QDBusIntrospection::Arguments inArguments; QDBusIntrospection::Annotations annotations; - while (xml.readNextStartElement()) { - if (xml.name() == QLatin1String("annotation")) { - parseAnnotation(xml, annotations); - } else if (xml.name() == QLatin1String("arg")) { - const QXmlStreamAttributes attributes = xml.attributes(); - const QString direction = attributes.value(QLatin1String("direction")).toString(); - QDBusIntrospection::Argument argument; - if (!attributes.hasAttribute(QLatin1String("direction")) - || direction == QLatin1String("in")) { - parseArg(attributes, argument); - inArguments << argument; - } else if (direction == QLatin1String("out")) { - parseArg(attributes, argument); - outArguments << argument; + if (!xml.readNextStartElement()) { + ifaceData->introspection += QLatin1String("/>\n"); + } else { + ifaceData->introspection += QLatin1String(">\n"); + + do { + if (xml.name() == QLatin1String("annotation")) { + parseAnnotation(xml, annotations, ifaceData); + } else if (xml.name() == QLatin1String("arg")) { + const QXmlStreamAttributes attributes = xml.attributes(); + const QString direction = attributes.value(QLatin1String("direction")).toString(); + QDBusIntrospection::Argument argument; + if (!attributes.hasAttribute(QLatin1String("direction")) + || direction == QLatin1String("in")) { + parseArg(attributes, argument, ifaceData); + inArguments << argument; + } else if (direction == QLatin1String("out")) { + parseArg(attributes, argument, ifaceData); + outArguments << argument; + } + } else if (xml.prefix().isEmpty()) { + qDBusParserError() << "Unknown element" << xml.name() << "while checking for method arguments"; } - } else if (xml.prefix().isEmpty()) { - qDBusParserError() << "Unknown element" << xml.name() << "while checking for method arguments"; - } - xml.skipCurrentElement(); + xml.skipCurrentElement(); + } while (xml.readNextStartElement()); + + ifaceData->introspection += QLatin1String(" </method>\n"); } methodData.inputArgs = inArguments; @@ -196,7 +231,7 @@ static bool parseMethod(QXmlStreamReader &xml, QDBusIntrospection::Method &metho static bool parseSignal(QXmlStreamReader &xml, QDBusIntrospection::Signal &signalData, - const QString &ifaceName) + QDBusIntrospection::Interface *ifaceData) { Q_ASSERT(xml.isStartElement() && xml.name() == QLatin1String("signal")); @@ -205,31 +240,39 @@ static bool parseSignal(QXmlStreamReader &xml, QDBusIntrospection::Signal &signa if (!QDBusUtil::isValidMemberName(signalName)) { qDBusParserError("Invalid D-BUS member name '%s' found in interface '%s' while parsing introspection", - qPrintable(signalName), qPrintable(ifaceName)); + qPrintable(signalName), qPrintable(ifaceData->name)); return false; } signalData.name = signalName; - + ifaceData->introspection += QLatin1String(" <signal name=\"") + signalName + QLatin1String("\""); QDBusIntrospection::Arguments arguments; QDBusIntrospection::Annotations annotations; - while (xml.readNextStartElement()) { - if (xml.name() == QLatin1String("annotation")) { - parseAnnotation(xml, annotations); - } else if (xml.name() == QLatin1String("arg")) { - const QXmlStreamAttributes attributes = xml.attributes(); - QDBusIntrospection::Argument argument; - if (!attributes.hasAttribute(QLatin1String("direction")) || - attributes.value(QLatin1String("direction")) == QLatin1String("out")) { - parseArg(attributes, argument); - arguments << argument; + if (!xml.readNextStartElement()) { + ifaceData->introspection += QLatin1String("/>\n"); + } else { + ifaceData->introspection += QLatin1String(">\n"); + + do { + if (xml.name() == QLatin1String("annotation")) { + parseAnnotation(xml, annotations, ifaceData); + } else if (xml.name() == QLatin1String("arg")) { + const QXmlStreamAttributes attributes = xml.attributes(); + QDBusIntrospection::Argument argument; + if (!attributes.hasAttribute(QLatin1String("direction")) || + attributes.value(QLatin1String("direction")) == QLatin1String("out")) { + parseArg(attributes, argument, ifaceData); + arguments << argument; + } + } else { + qDBusParserError() << "Unknown element" << xml.name() << "while checking for signal arguments"; } - } else { - qDBusParserError() << "Unknown element" << xml.name() << "while checking for signal arguments"; - } - xml.skipCurrentElement(); + xml.skipCurrentElement(); + } while (xml.readNextStartElement()); + + ifaceData->introspection += QLatin1String(" </signal>\n"); } signalData.outputArgs = arguments; @@ -252,22 +295,23 @@ static void readInterface(QXmlStreamReader &xml, QDBusIntrospection::Object *obj QDBusIntrospection::Interface *ifaceData = new QDBusIntrospection::Interface; ifaceData->name = ifaceName; + ifaceData->introspection += QLatin1String(" <interface name=\"") + ifaceName + QLatin1String("\">\n"); while (xml.readNextStartElement()) { if (xml.name() == QLatin1String("method")) { QDBusIntrospection::Method methodData; - if (parseMethod(xml, methodData, ifaceName)) + if (parseMethod(xml, methodData, ifaceData)) ifaceData->methods.insert(methodData.name, methodData); } else if (xml.name() == QLatin1String("signal")) { QDBusIntrospection::Signal signalData; - if (parseSignal(xml, signalData, ifaceName)) + if (parseSignal(xml, signalData, ifaceData)) ifaceData->signals_.insert(signalData.name, signalData); } else if (xml.name() == QLatin1String("property")) { QDBusIntrospection::Property propertyData; - if (parseProperty(xml, propertyData, ifaceName)) + if (parseProperty(xml, propertyData, ifaceData)) ifaceData->properties.insert(propertyData.name, propertyData); } else if (xml.name() == QLatin1String("annotation")) { - parseAnnotation(xml, ifaceData->annotations); + parseAnnotation(xml, ifaceData->annotations, ifaceData, true); xml.skipCurrentElement(); // skip over annotation object } else { if (xml.prefix().isEmpty()) { @@ -277,6 +321,8 @@ static void readInterface(QXmlStreamReader &xml, QDBusIntrospection::Object *obj } } + ifaceData->introspection += QLatin1String(" </interface>"); + interfaces->insert(ifaceName, QSharedDataPointer<QDBusIntrospection::Interface>(ifaceData)); if (!xml.isEndElement() || xml.name() != QLatin1String("interface")) { |