Differential D26202 Diff 73608 src/kconfig_compiler/KCFGXmlParser.h

Changeset View

Standalone View

View Options

src/kconfig_compiler/KCFGXmlParser.h

  • This file was added.
1​/* This file is part of the KDE libraries
2​ Copyright (c) 2003 Cornelius Schumacher <schumacher@kde.org>
dfaureUnsubmitted
Done

missing license header

(repeats)

dfaure: missing license header (repeats)
3​ Copyright (c) 2003 Waldo Bastian <bastian@kde.org>
4​ Copyright (c) 2003 Zack Rusin <zack@kde.org>
5​ Copyright (c) 2006 Michaël Larouche <michael.larouche@kdemail.net>
6​ Copyright (c) 2008 Allen Winter <winter@kde.org>
7​ Copyright (C) 2020 Tomaz Cananbrava (tcanabrava@kde.org)
8​
9​ This library is free software; you can redistribute it and/or
10​ modify it under the terms of the GNU Library General Public
11​ License as published by the Free Software Foundation; either
12​ version 2 of the License, or (at your option) any later version.
13​
14​ This library is distributed in the hope that it will be useful,
15​ but WITHOUT ANY WARRANTY; without even the implied warranty of
16​ MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
dfaureUnsubmitted
Done

... const;

dfaure: ... const;
17​ Library General Public License for more details.
dfaureUnsubmitted
Done

... const;

dfaure: ... const;
18​
19​ You should have received a copy of the GNU Library General Public License
20​ along with this library; see the file COPYING.LIB. If not, write to
21​ the Free Software Foundation, Inc., 51 Franklin Street, Fifth Floor,
22​ Boston, MA 02110-1301, USA.
23​*/
24​
25​#ifndef KCFGXMLPARSER_H
26​#define KCFGXMLPARSER_H
27​
28​#include <QDomDocument>
29​#include <QString>
dfaureUnsubmitted
Done

(to pick on one random method as an example) : this is only used internally by the class, it should be made private. Please do this for all methods that are not used from outside the class. It will make it much clearer what's the actual API of the class.

dfaure: (to pick on one random method as an example) : this is only used internally by the class, it…
30​#include <QRegularExpression>
31​
32​#include "KConfigCommonStructs.h"
33​#include "KConfigXTParameters.h"
34​
35​/* This parses the contents of a Xml file into a ParseResult Structure,
dfaureUnsubmitted
Done

how, why, when?

dfaure: how, why, when?
36​ * It also fails hard:
37​ * If start() succeeds, you can use the result,
38​ * if start() fails, the program aborts with an error message so there's
39​ * no possibility of generating incorrect code information.
40​ */
41​class KCFGXmlParser {
42​public:
ervinUnsubmitted
Done

Please put the opening curly brace on its own line.

As far as the class name goes I'd propose KCfgXmlParser or KConfigXmlParser to make sure we have a consistent capitalization.

ervin: Please put the opening curly brace on its own line. As far as the class name goes I'd propose…
43​ KCFGXmlParser(const KConfigXTParameters &cfg, const QString& inputFileName);
44​
ervinUnsubmitted
Done

Please have the space before & and not after.

ervin: Please have the space before & and not after.
45​ // Start the parser and reads the contents of the inputFileName into the ParseResult Structure
46​ void start();
47​
48​ // Get the result of the parse
dfaureUnsubmitted
Done

typo ("something")

dfaure: typo ("something")
49​ ParseResult getParseResult() const;
50​
51​private:
52​ // creates a `somethingChanged` signal for every property
53​ void createChangedSignal(CfgEntry &readEntry);
54​
55​ void validateNameAndKey(CfgEntry &readEntry, const QDomElement &element);
56​
57​ // TODO: Use std::optional and CfgEntry (without heap allocation) for this function
58​ // *or* fail hard if the parse fails.
59​ CfgEntry *parseEntry(const QString &group, const QDomElement &element);
60​
61​ // Steps
62​ void readIncludeTag(const QDomElement &element);
63​ void readGroupTag(const QDomElement &element);
64​ void readKcfgfileTag(const QDomElement &element);
65​ void readSignalTag(const QDomElement &element);
66​
67​ // Those are the Entries in the Xml, that represent a parameter within the <group> </group> tag.
68​ void readParameterFromEntry(CfgEntry &entry, const QDomElement &element);
69​ bool hasDefaultCode(CfgEntry &entry, const QDomElement &element);
70​ void readChoicesFromEntry(CfgEntry &entry, const QDomElement &element);
71​ void readGroupElements(CfgEntry &entry, const QDomElement &element);
72​ void readParamDefaultValues(CfgEntry &entry, const QDomElement &element);
73​
74​private:
75​ ParseResult mParseResult;
76​ KConfigXTParameters cfg;
77​ QString mInputFileName;
78​ QStringList mAllNames;
79​ QRegularExpression mValidNameRegexp;
80​};
81​
82​#endif
dfaureUnsubmitted
Done

should be fixed

dfaure: should be fixed