[KConfigLoader] Code cleanup

Summary:
- Drop redundant args to startElement()/endEelement()
- Use range-for
- Drop ConfigLoaderHandler methods that weren't used anywhere AFAICS
  and ConfigLoaderHandler is private API

Test Plan: make && ctest

Reviewers: #frameworks, apol, dfaure

Reviewed By: dfaure

Subscribers: kde-frameworks-devel

Tags: #frameworks

Differential Revision: https://phabricator.kde.org/D27157

1 files changed, 2 insertions, 11 deletions

diff --git a/src/gui/kconfigloaderhandler_p.h b/src/gui/kconfigloaderhandler_p.h
index fa313f5b..18e9622a 100644
--- a/src/gui/kconfigloaderhandler_p.h
+++ b/src/gui/kconfigloaderhandler_p.h
@@ -30,19 +30,10 @@ public:
 
     bool parse(QIODevice *input);
 
-    bool startElement(const QStringRef &namespaceURI, const QStringRef &localName,
-                      const QStringRef &qName, const QXmlStreamAttributes &atts);
-    bool endElement(const QStringRef &namespaceURI, const QStringRef &localName,
-                    const QStringRef &qName);
+    bool startElement(const QStringRef &localName, const QXmlStreamAttributes &attrs);
+    bool endElement(const QStringRef &localName);
     bool characters(const QStringRef &ch);
 
-    QString name() const;
-    void setName(const QString &name);
-    QString key() const;
-    void setKey(const QString &name);
-    QString type() const;
-    QString defaultValue() const;
-
 private:
     void addItem();
     void resetState();