From 783d1c28c88229f81715f6ecd1cf3866855252a2 Mon Sep 17 00:00:00 2001 From: Ahmad Samir Date: Fri, 13 Aug 2021 02:10:47 +0200 Subject: clang-tidy: one declaration per line; braces around statements clang-tidy checks: readability-isolate-declaration and readability-braces-around-statements KF task: https://phabricator.kde.org/T14729 GIT_SILENT --- src/kreadconfig/kreadconfig.cpp | 16 ++++++++++------ 1 file changed, 10 insertions(+), 6 deletions(-) (limited to 'src/kreadconfig/kreadconfig.cpp') diff --git a/src/kreadconfig/kreadconfig.cpp b/src/kreadconfig/kreadconfig.cpp index f0fb3518..01d44326 100644 --- a/src/kreadconfig/kreadconfig.cpp +++ b/src/kreadconfig/kreadconfig.cpp @@ -67,9 +67,9 @@ int main(int argc, char **argv) KConfig *konfig; bool configMustDeleted = false; - if (file.isEmpty()) + if (file.isEmpty()) { konfig = KSharedConfig::openConfig().data(); - else { + } else { konfig = new KConfig(file, KConfig::NoGlobals); configMustDeleted = true; } @@ -82,24 +82,28 @@ int main(int argc, char **argv) dflt = dflt.toLower(); bool def = (dflt == QLatin1String{"true"} || dflt == QLatin1String{"on"} || dflt == QLatin1String{"yes"} || dflt == QLatin1String{"1"}); bool retValue = !cfgGroup.readEntry(key, def); - if (configMustDeleted) + if (configMustDeleted) { delete konfig; + } return retValue; } else if (type == QLatin1String{"num"} || type == QLatin1String{"int"}) { int retValue = cfgGroup.readEntry(key, dflt.toInt()); - if (configMustDeleted) + if (configMustDeleted) { delete konfig; + } return retValue; } else if (type == QLatin1String{"path"}) { fprintf(stdout, "%s\n", cfgGroup.readPathEntry(key, dflt).toLocal8Bit().data()); - if (configMustDeleted) + if (configMustDeleted) { delete konfig; + } return 0; } else { /* Assume it's a string... */ fprintf(stdout, "%s\n", cfgGroup.readEntry(key, dflt).toLocal8Bit().data()); - if (configMustDeleted) + if (configMustDeleted) { delete konfig; + } return 0; } } -- cgit v1.2.1