D13719: OdfExtractor: deal with non-common prefix names
Jos van den Oever
noreply at phabricator.kde.org
Mon Jun 25 17:30:39 UTC 2018
vandenoever accepted this revision.
vandenoever added a comment.
This revision is now accepted and ready to land.
Good fixes. Just some small naming suggestions.
INLINE COMMENTS
> odfextractor.cpp:37
> +
> +QDomElement namedItemNS(const QDomNode &node, const QString &nsURI, const QString &localName)
> +{
Perhaps name this `elementByTagNameNS`.
> odfextractor.cpp:39
> +{
> + for (auto n = node.firstChildElement(); !n.isNull(); n = n.nextSiblingElement()) {
> + if (n.localName() == localName && n.namespaceURI() == nsURI) {
Variable name `e` for element is more logical.
REPOSITORY
R286 KFileMetaData
BRANCH
noncommonprefixodf
REVISION DETAIL
https://phabricator.kde.org/D13719
To: kossebau, vandenoever
Cc: kde-frameworks-devel, #baloo, ashaposhnikov, michaelh, astippich, spoorun, ngraham, bruns, abrahams
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20180625/b3124248/attachment-0001.html>
More information about the Kde-frameworks-devel
mailing list