aboutsummaryrefslogtreecommitdiffstats
path: root/src/plugins/beautifier
diff options
context:
space:
mode:
authorhjk <hjk@qt.io>2020-06-17 08:33:53 +0200
committerhjk <hjk@qt.io>2020-06-19 11:42:10 +0000
commitc9a7a3a2d30259adff4dd3ed81b6002dcbc2e875 (patch)
tree7d9ec468d350549f42c1e4ace313f02d71c63e85 /src/plugins/beautifier
parente064100007c34cf6995de111c2c929eeda852f91 (diff)
Beautifier: Remove inner namespace level
And inline non-shared object id. I regularly stumble over this speciality when doing mass refactorings in the code base and I believe the extra benefit this namespace level has does not outweigh it. Change-Id: I3694a656f1095dcb5509b212b9e1d25ae5c8b5ec Reviewed-by: Christian Stenger <christian.stenger@qt.io>
Diffstat (limited to 'src/plugins/beautifier')
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyle.cpp11
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyle.h2
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyleconstants.h8
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.cpp6
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.h2
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.ui4
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstylesettings.cpp9
-rw-r--r--src/plugins/beautifier/artisticstyle/artisticstylesettings.h2
-rw-r--r--src/plugins/beautifier/beautifierplugin.cpp6
-rw-r--r--src/plugins/beautifier/clangformat/clangformat.cpp18
-rw-r--r--src/plugins/beautifier/clangformat/clangformat.h2
-rw-r--r--src/plugins/beautifier/clangformat/clangformatconstants.h10
-rw-r--r--src/plugins/beautifier/clangformat/clangformatoptionspage.cpp4
-rw-r--r--src/plugins/beautifier/clangformat/clangformatoptionspage.h2
-rw-r--r--src/plugins/beautifier/clangformat/clangformatoptionspage.ui4
-rw-r--r--src/plugins/beautifier/clangformat/clangformatsettings.cpp9
-rw-r--r--src/plugins/beautifier/clangformat/clangformatsettings.h2
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustify.cpp16
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustify.h2
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifyconstants.h9
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifyoptionspage.cpp6
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifyoptionspage.h2
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifyoptionspage.ui4
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifysettings.cpp9
-rw-r--r--src/plugins/beautifier/uncrustify/uncrustifysettings.h2
25 files changed, 44 insertions, 107 deletions
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyle.cpp b/src/plugins/beautifier/artisticstyle/artisticstyle.cpp
index 6926ce8843..a8851b43cc 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyle.cpp
+++ b/src/plugins/beautifier/artisticstyle/artisticstyle.cpp
@@ -58,16 +58,14 @@ using namespace TextEditor;
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
ArtisticStyle::ArtisticStyle()
{
- Core::ActionContainer *menu = Core::ActionManager::createMenu(Constants::ArtisticStyle::MENU_ID);
+ Core::ActionContainer *menu = Core::ActionManager::createMenu("ArtisticStyle.Menu");
menu->menu()->setTitle(tr("&Artistic Style"));
m_formatFile = new QAction(BeautifierPlugin::msgFormatCurrentFile(), this);
- menu->addAction(Core::ActionManager::registerAction(m_formatFile,
- Constants::ArtisticStyle::ACTION_FORMATFILE));
+ menu->addAction(Core::ActionManager::registerAction(m_formatFile, "ArtisticStyle.FormatFile"));
connect(m_formatFile, &QAction::triggered, this, &ArtisticStyle::formatFile);
Core::ActionManager::actionContainer(Constants::MENU_ID)->addMenu(menu);
@@ -78,7 +76,7 @@ ArtisticStyle::ArtisticStyle()
QString ArtisticStyle::id() const
{
- return QLatin1String(Constants::ArtisticStyle::DISPLAY_NAME);
+ return QLatin1String(Constants::ARTISTICSTYLE_DISPLAY_NAME);
}
void ArtisticStyle::updateActions(Core::IEditor *editor)
@@ -91,7 +89,7 @@ void ArtisticStyle::formatFile()
const QString cfgFileName = configurationFile();
if (cfgFileName.isEmpty()) {
BeautifierPlugin::showError(BeautifierPlugin::msgCannotGetConfigurationFile(
- tr(Constants::ArtisticStyle::DISPLAY_NAME)));
+ tr(Constants::ARTISTICSTYLE_DISPLAY_NAME)));
} else {
formatCurrentFile(command(cfgFileName));
}
@@ -166,6 +164,5 @@ Command ArtisticStyle::command(const QString &cfgFile) const
return command;
}
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyle.h b/src/plugins/beautifier/artisticstyle/artisticstyle.h
index 6844f65b08..56eaeb5116 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyle.h
+++ b/src/plugins/beautifier/artisticstyle/artisticstyle.h
@@ -32,7 +32,6 @@
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
class ArtisticStyle : public BeautifierAbstractTool
{
@@ -56,6 +55,5 @@ private:
ArtisticStyleOptionsPage m_page{&m_settings};
};
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyleconstants.h b/src/plugins/beautifier/artisticstyle/artisticstyleconstants.h
index 3916cfa436..52bf88cc20 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyleconstants.h
+++ b/src/plugins/beautifier/artisticstyle/artisticstyleconstants.h
@@ -29,14 +29,8 @@
namespace Beautifier {
namespace Constants {
-namespace ArtisticStyle {
-const char DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal::ArtisticStyle::ArtisticStyle", "Artistic Style");
-const char ACTION_FORMATFILE[] = "ArtisticStyle.FormatFile";
-const char MENU_ID[] = "ArtisticStyle.Menu";
-const char OPTION_ID[] = "ArtisticStyle";
-const char SETTINGS_NAME[] = "artisticstyle";
+const char ARTISTICSTYLE_DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal", "Artistic Style");
-} // namespace ArtisticStyle
} // namespace Constants
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.cpp b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.cpp
index 922f747742..729c44c15b 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.cpp
+++ b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.cpp
@@ -35,7 +35,6 @@
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
class ArtisticStyleOptionsPageWidget : public Core::IOptionsPageWidget
{
@@ -62,7 +61,7 @@ ArtisticStyleOptionsPageWidget::ArtisticStyleOptionsPageWidget(ArtisticStyleSett
ui.command->setExpectedKind(Utils::PathChooser::ExistingCommand);
ui.command->setCommandVersionArguments({"--version"});
ui.command->setPromptDialogTitle(BeautifierPlugin::msgCommandPromptDialogTitle(
- ArtisticStyle::tr(Constants::ArtisticStyle::DISPLAY_NAME)));
+ ArtisticStyle::tr(Constants::ARTISTICSTYLE_DISPLAY_NAME)));
connect(ui.command, &Utils::PathChooser::validChanged, ui.options, &QWidget::setEnabled);
ui.configurations->setSettings(m_settings);
@@ -94,12 +93,11 @@ void ArtisticStyleOptionsPageWidget::apply()
ArtisticStyleOptionsPage::ArtisticStyleOptionsPage(ArtisticStyleSettings *settings)
{
- setId(Constants::ArtisticStyle::OPTION_ID);
+ setId("ArtisticStyle");
setDisplayName(ArtisticStyleOptionsPageWidget::tr("Artistic Style"));
setCategory(Constants::OPTION_CATEGORY);
setWidgetCreator([settings] { return new ArtisticStyleOptionsPageWidget(settings); });
}
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.h b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.h
index e31182ebba..eda00f020a 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.h
+++ b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.h
@@ -29,7 +29,6 @@
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
class ArtisticStyleSettings;
@@ -39,6 +38,5 @@ public:
explicit ArtisticStyleOptionsPage(ArtisticStyleSettings *settings);
};
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.ui b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.ui
index 06f5cfc3e4..127f37105f 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.ui
+++ b/src/plugins/beautifier/artisticstyle/artisticstyleoptionspage.ui
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<ui version="4.0">
- <class>Beautifier::Internal::ArtisticStyle::ArtisticStyleOptionsPage</class>
- <widget class="QWidget" name="Beautifier::Internal::ArtisticStyle::ArtisticStyleOptionsPage">
+ <class>Beautifier::Internal::ArtisticStyleOptionsPage</class>
+ <widget class="QWidget" name="Beautifier::Internal::ArtisticStyleOptionsPage">
<property name="geometry">
<rect>
<x>0</x>
diff --git a/src/plugins/beautifier/artisticstyle/artisticstylesettings.cpp b/src/plugins/beautifier/artisticstyle/artisticstylesettings.cpp
index dfb2c57f91..9a78461017 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstylesettings.cpp
+++ b/src/plugins/beautifier/artisticstyle/artisticstylesettings.cpp
@@ -43,19 +43,17 @@
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
-namespace {
const char USE_OTHER_FILES[] = "useOtherFiles";
const char USE_SPECIFIC_CONFIG_FILE[] = "useSpecificConfigFile";
const char SPECIFIC_CONFIG_FILE[] = "specificConfigFile";
const char USE_HOME_FILE[] = "useHomeFile";
const char USE_CUSTOM_STYLE[] = "useCustomStyle";
const char CUSTOM_STYLE[] = "customStyle";
-}
+const char SETTINGS_NAME[] = "artisticstyle";
ArtisticStyleSettings::ArtisticStyleSettings() :
- AbstractSettings(Constants::ArtisticStyle::SETTINGS_NAME, ".astyle")
+ AbstractSettings(SETTINGS_NAME, ".astyle")
{
connect(&m_versionWatcher, &QFutureWatcherBase::finished,
this, &ArtisticStyleSettings::helperSetVersion);
@@ -175,7 +173,7 @@ QString ArtisticStyleSettings::documentationFilePath() const
{
return Core::ICore::userResourcePath() + '/' + Beautifier::Constants::SETTINGS_DIRNAME + '/'
+ Beautifier::Constants::DOCUMENTATION_DIRNAME + '/'
- + Constants::ArtisticStyle::SETTINGS_NAME + ".xml";
+ + SETTINGS_NAME + ".xml";
}
void ArtisticStyleSettings::createDocumentationFile() const
@@ -250,6 +248,5 @@ void ArtisticStyleSettings::createDocumentationFile() const
}
}
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/artisticstyle/artisticstylesettings.h b/src/plugins/beautifier/artisticstyle/artisticstylesettings.h
index 50deba87d6..e0fa1b8866 100644
--- a/src/plugins/beautifier/artisticstyle/artisticstylesettings.h
+++ b/src/plugins/beautifier/artisticstyle/artisticstylesettings.h
@@ -34,7 +34,6 @@
namespace Beautifier {
namespace Internal {
-namespace ArtisticStyle {
class ArtisticStyleSettings : public AbstractSettings
{
@@ -77,6 +76,5 @@ private:
QFutureWatcher<int> m_versionWatcher;
};
-} // namespace ArtisticStyle
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/beautifierplugin.cpp b/src/plugins/beautifier/beautifierplugin.cpp
index fa74da7e26..661b41d016 100644
--- a/src/plugins/beautifier/beautifierplugin.cpp
+++ b/src/plugins/beautifier/beautifierplugin.cpp
@@ -98,9 +98,9 @@ public:
GeneralSettings generalSettings;
- ArtisticStyle::ArtisticStyle artisticStyleBeautifier;
- ClangFormat::ClangFormat clangFormatBeautifier;
- Uncrustify::Uncrustify uncrustifyBeautifier;
+ ArtisticStyle artisticStyleBeautifier;
+ ClangFormat clangFormatBeautifier;
+ Uncrustify uncrustifyBeautifier;
BeautifierAbstractTool *m_tools[3] {
&artisticStyleBeautifier,
diff --git a/src/plugins/beautifier/clangformat/clangformat.cpp b/src/plugins/beautifier/clangformat/clangformat.cpp
index 16aee422c2..10d32566cf 100644
--- a/src/plugins/beautifier/clangformat/clangformat.cpp
+++ b/src/plugins/beautifier/clangformat/clangformat.cpp
@@ -53,35 +53,34 @@ using namespace TextEditor;
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
+
+const char MENU_ID[] = "ClangFormat.Menu";
ClangFormat::ClangFormat()
{
- Core::ActionContainer *menu = Core::ActionManager::createMenu(Constants::ClangFormat::MENU_ID);
+ Core::ActionContainer *menu = Core::ActionManager::createMenu(MENU_ID);
menu->menu()->setTitle(tr("&ClangFormat"));
m_formatFile = new QAction(BeautifierPlugin::msgFormatCurrentFile(), this);
Core::Command *cmd
- = Core::ActionManager::registerAction(m_formatFile,
- Constants::ClangFormat::ACTION_FORMATFILE);
+ = Core::ActionManager::registerAction(m_formatFile, "ClangFormat.FormatFile");
menu->addAction(cmd);
connect(m_formatFile, &QAction::triggered, this, &ClangFormat::formatFile);
m_formatRange = new QAction(BeautifierPlugin::msgFormatAtCursor(), this);
- cmd = Core::ActionManager::registerAction(m_formatRange,
- Constants::ClangFormat::ACTION_FORMATATCURSOR);
+ cmd = Core::ActionManager::registerAction(m_formatRange, "ClangFormat.FormatAtCursor");
menu->addAction(cmd);
connect(m_formatRange, &QAction::triggered, this, &ClangFormat::formatAtCursor);
m_disableFormattingSelectedText
= new QAction(BeautifierPlugin::msgDisableFormattingSelectedText(), this);
cmd = Core::ActionManager::registerAction(
- m_disableFormattingSelectedText, Constants::ClangFormat::ACTION_DISABLEFORMATTINGSELECTED);
+ m_disableFormattingSelectedText, "ClangFormat.DisableFormattingSelectedText");
menu->addAction(cmd);
connect(m_disableFormattingSelectedText, &QAction::triggered,
this, &ClangFormat::disableFormattingSelectedText);
- Core::ActionManager::actionContainer(Constants::MENU_ID)->addMenu(menu);
+ Core::ActionManager::actionContainer(MENU_ID)->addMenu(menu);
connect(&m_settings, &ClangFormatSettings::supportedMimeTypesChanged,
[this] { updateActions(Core::EditorManager::currentEditor()); });
@@ -89,7 +88,7 @@ ClangFormat::ClangFormat()
QString ClangFormat::id() const
{
- return QLatin1String(Constants::ClangFormat::DISPLAY_NAME);
+ return QLatin1String(Constants::CLANGFORMAT_DISPLAY_NAME);
}
void ClangFormat::updateActions(Core::IEditor *editor)
@@ -200,6 +199,5 @@ Command ClangFormat::command(int offset, int length) const
return c;
}
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformat.h b/src/plugins/beautifier/clangformat/clangformat.h
index 83252e99a0..3207f04349 100644
--- a/src/plugins/beautifier/clangformat/clangformat.h
+++ b/src/plugins/beautifier/clangformat/clangformat.h
@@ -32,7 +32,6 @@
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
class ClangFormat : public BeautifierAbstractTool
{
@@ -59,6 +58,5 @@ private:
ClangFormatOptionsPage m_page{&m_settings};
};
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformatconstants.h b/src/plugins/beautifier/clangformat/clangformatconstants.h
index 2c94b347a5..f71203197e 100644
--- a/src/plugins/beautifier/clangformat/clangformatconstants.h
+++ b/src/plugins/beautifier/clangformat/clangformatconstants.h
@@ -29,16 +29,8 @@
namespace Beautifier {
namespace Constants {
-namespace ClangFormat {
-const char DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal::ClangFormat::ClangFormat", "ClangFormat");
-const char ACTION_FORMATFILE[] = "ClangFormat.FormatFile";
-const char ACTION_FORMATATCURSOR[] = "ClangFormat.FormatAtCursor";
-const char ACTION_DISABLEFORMATTINGSELECTED[] = "ClangFormat.DisableFormattingSelectedText";
-const char MENU_ID[] = "ClangFormat.Menu";
-const char OPTION_ID[] = "ClangFormat";
-const char SETTINGS_NAME[] = "clangformat";
+const char CLANGFORMAT_DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal::ClangFormat", "ClangFormat");
-} // namespace ClangFormat
} // namespace Constants
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformatoptionspage.cpp b/src/plugins/beautifier/clangformat/clangformatoptionspage.cpp
index 767b44064c..6a8e250353 100644
--- a/src/plugins/beautifier/clangformat/clangformatoptionspage.cpp
+++ b/src/plugins/beautifier/clangformat/clangformatoptionspage.cpp
@@ -34,7 +34,6 @@
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
class ClangFormatOptionsPageWidget : public Core::IOptionsPageWidget
{
@@ -104,12 +103,11 @@ void ClangFormatOptionsPageWidget::apply()
ClangFormatOptionsPage::ClangFormatOptionsPage(ClangFormatSettings *settings)
{
- setId(Constants::ClangFormat::OPTION_ID);
+ setId("ClangFormat");
setDisplayName(ClangFormatOptionsPageWidget::tr("Clang Format"));
setCategory(Constants::OPTION_CATEGORY);
setWidgetCreator([settings] { return new ClangFormatOptionsPageWidget(settings); });
}
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformatoptionspage.h b/src/plugins/beautifier/clangformat/clangformatoptionspage.h
index 5e5013feef..ac2621cbf2 100644
--- a/src/plugins/beautifier/clangformat/clangformatoptionspage.h
+++ b/src/plugins/beautifier/clangformat/clangformatoptionspage.h
@@ -29,7 +29,6 @@
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
class ClangFormatSettings;
@@ -39,6 +38,5 @@ public:
explicit ClangFormatOptionsPage(ClangFormatSettings *settings);
};
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformatoptionspage.ui b/src/plugins/beautifier/clangformat/clangformatoptionspage.ui
index 06f6ef84b2..1cf89000a4 100644
--- a/src/plugins/beautifier/clangformat/clangformatoptionspage.ui
+++ b/src/plugins/beautifier/clangformat/clangformatoptionspage.ui
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<ui version="4.0">
- <class>Beautifier::Internal::ClangFormat::ClangFormatOptionsPage</class>
- <widget class="QWidget" name="Beautifier::Internal::ClangFormat::ClangFormatOptionsPage">
+ <class>Beautifier::Internal::ClangFormatOptionsPage</class>
+ <widget class="QWidget" name="Beautifier::Internal::ClangFormatOptionsPage">
<property name="geometry">
<rect>
<x>0</x>
diff --git a/src/plugins/beautifier/clangformat/clangformatsettings.cpp b/src/plugins/beautifier/clangformat/clangformatsettings.cpp
index 6369c60510..d2b5416715 100644
--- a/src/plugins/beautifier/clangformat/clangformatsettings.cpp
+++ b/src/plugins/beautifier/clangformat/clangformatsettings.cpp
@@ -36,17 +36,15 @@
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
-namespace {
const char USE_PREDEFINED_STYLE[] = "usePredefinedStyle";
const char PREDEFINED_STYLE[] = "predefinedStyle";
const char FALLBACK_STYLE[] = "fallbackStyle";
const char CUSTOM_STYLE[] = "customStyle";
-}
+const char SETTINGS_NAME[] = "clangformat";
ClangFormatSettings::ClangFormatSettings() :
- AbstractSettings(Constants::ClangFormat::SETTINGS_NAME, ".clang-format")
+ AbstractSettings(SETTINGS_NAME, ".clang-format")
{
setCommand("clang-format");
m_settings.insert(USE_PREDEFINED_STYLE, QVariant(true));
@@ -60,7 +58,7 @@ QString ClangFormatSettings::documentationFilePath() const
{
return Core::ICore::userResourcePath() + '/' + Beautifier::Constants::SETTINGS_DIRNAME + '/'
+ Beautifier::Constants::DOCUMENTATION_DIRNAME + '/'
- + Constants::ClangFormat::SETTINGS_NAME + ".xml";
+ + SETTINGS_NAME + ".xml";
}
void ClangFormatSettings::createDocumentationFile() const
@@ -238,6 +236,5 @@ void ClangFormatSettings::readStyles()
}
}
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/clangformat/clangformatsettings.h b/src/plugins/beautifier/clangformat/clangformatsettings.h
index 4fca7ddc24..079a643865 100644
--- a/src/plugins/beautifier/clangformat/clangformatsettings.h
+++ b/src/plugins/beautifier/clangformat/clangformatsettings.h
@@ -29,7 +29,6 @@
namespace Beautifier {
namespace Internal {
-namespace ClangFormat {
class ClangFormatSettings : public AbstractSettings
{
@@ -63,6 +62,5 @@ private:
void readStyles() override;
};
-} // namespace ClangFormat
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustify.cpp b/src/plugins/beautifier/uncrustify/uncrustify.cpp
index c8069a9df9..689ef3714a 100644
--- a/src/plugins/beautifier/uncrustify/uncrustify.cpp
+++ b/src/plugins/beautifier/uncrustify/uncrustify.cpp
@@ -54,23 +54,20 @@ using namespace TextEditor;
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
Uncrustify::Uncrustify()
{
- Core::ActionContainer *menu = Core::ActionManager::createMenu(Constants::Uncrustify::MENU_ID);
+ Core::ActionContainer *menu = Core::ActionManager::createMenu("Uncrustify.Menu");
menu->menu()->setTitle(tr("&Uncrustify"));
m_formatFile = new QAction(BeautifierPlugin::msgFormatCurrentFile(), this);
Core::Command *cmd
- = Core::ActionManager::registerAction(m_formatFile,
- Constants::Uncrustify::ACTION_FORMATFILE);
+ = Core::ActionManager::registerAction(m_formatFile, "Uncrustify.FormatFile");
menu->addAction(cmd);
connect(m_formatFile, &QAction::triggered, this, &Uncrustify::formatFile);
m_formatRange = new QAction(BeautifierPlugin::msgFormatSelectedText(), this);
- cmd = Core::ActionManager::registerAction(m_formatRange,
- Constants::Uncrustify::ACTION_FORMATSELECTED);
+ cmd = Core::ActionManager::registerAction(m_formatRange, "Uncrustify.FormatSelectedText");
menu->addAction(cmd);
connect(m_formatRange, &QAction::triggered, this, &Uncrustify::formatSelectedText);
@@ -82,7 +79,7 @@ Uncrustify::Uncrustify()
QString Uncrustify::id() const
{
- return QLatin1String(Constants::Uncrustify::DISPLAY_NAME);
+ return QLatin1String(Constants::UNCRUSTIFY_DISPLAY_NAME);
}
void Uncrustify::updateActions(Core::IEditor *editor)
@@ -97,7 +94,7 @@ void Uncrustify::formatFile()
const QString cfgFileName = configurationFile();
if (cfgFileName.isEmpty()) {
BeautifierPlugin::showError(BeautifierPlugin::msgCannotGetConfigurationFile(
- tr(Constants::Uncrustify::DISPLAY_NAME)));
+ tr(Constants::UNCRUSTIFY_DISPLAY_NAME)));
} else {
formatCurrentFile(command(cfgFileName));
}
@@ -108,7 +105,7 @@ void Uncrustify::formatSelectedText()
const QString cfgFileName = configurationFile();
if (cfgFileName.isEmpty()) {
BeautifierPlugin::showError(BeautifierPlugin::msgCannotGetConfigurationFile(
- tr(Constants::Uncrustify::DISPLAY_NAME)));
+ tr(Constants::UNCRUSTIFY_DISPLAY_NAME)));
return;
}
@@ -199,6 +196,5 @@ Command Uncrustify::command(const QString &cfgFile, bool fragment) const
return command;
}
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustify.h b/src/plugins/beautifier/uncrustify/uncrustify.h
index a25d288d8c..4b97838a8a 100644
--- a/src/plugins/beautifier/uncrustify/uncrustify.h
+++ b/src/plugins/beautifier/uncrustify/uncrustify.h
@@ -32,7 +32,6 @@
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
class Uncrustify : public BeautifierAbstractTool
{
@@ -58,6 +57,5 @@ private:
UncrustifyOptionsPage m_page{&m_settings};
};
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustifyconstants.h b/src/plugins/beautifier/uncrustify/uncrustifyconstants.h
index 7fa9537c57..131f798a05 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifyconstants.h
+++ b/src/plugins/beautifier/uncrustify/uncrustifyconstants.h
@@ -29,15 +29,8 @@
namespace Beautifier {
namespace Constants {
-namespace Uncrustify {
-const char DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal::Uncrustify::Uncrustify", "Uncrustify");
-const char ACTION_FORMATFILE[] = "Uncrustify.FormatFile";
-const char ACTION_FORMATSELECTED[] = "Uncrustify.FormatSelectedText";
-const char MENU_ID[] = "Uncrustify.Menu";
-const char OPTION_ID[] = "Uncrustify";
-const char SETTINGS_NAME[] = "uncrustify";
+const char UNCRUSTIFY_DISPLAY_NAME[] = QT_TRANSLATE_NOOP("Beautifier::Internal", "Uncrustify");
-} // namespace Uncrustify
} // namespace Constants
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.cpp b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.cpp
index 10dd6377d8..2a93458108 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.cpp
+++ b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.cpp
@@ -35,7 +35,6 @@
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
class UncrustifyOptionsPageWidget : public Core::IOptionsPageWidget
{
@@ -63,7 +62,7 @@ UncrustifyOptionsPageWidget::UncrustifyOptionsPageWidget(UncrustifySettings *set
ui.command->setExpectedKind(Utils::PathChooser::ExistingCommand);
ui.command->setCommandVersionArguments({"--version"});
ui.command->setPromptDialogTitle(BeautifierPlugin::msgCommandPromptDialogTitle(
- Uncrustify::tr(Constants::Uncrustify::DISPLAY_NAME)));
+ Uncrustify::tr(Constants::UNCRUSTIFY_DISPLAY_NAME)));
connect(ui.command, &Utils::PathChooser::validChanged, ui.options, &QWidget::setEnabled);
ui.configurations->setSettings(m_settings);
@@ -97,12 +96,11 @@ void UncrustifyOptionsPageWidget::apply()
UncrustifyOptionsPage::UncrustifyOptionsPage(UncrustifySettings *settings)
{
- setId(Constants::Uncrustify::OPTION_ID);
+ setId("Uncrustify");
setDisplayName(UncrustifyOptionsPageWidget::tr("Uncrustify"));
setCategory(Constants::OPTION_CATEGORY);
setWidgetCreator([settings] { return new UncrustifyOptionsPageWidget(settings); });
}
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.h b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.h
index 04312b241d..37802b655e 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.h
+++ b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.h
@@ -29,7 +29,6 @@
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
class UncrustifySettings;
@@ -39,6 +38,5 @@ public:
explicit UncrustifyOptionsPage(UncrustifySettings *settings);
};
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.ui b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.ui
index 690044bc50..67a59edfa2 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.ui
+++ b/src/plugins/beautifier/uncrustify/uncrustifyoptionspage.ui
@@ -1,7 +1,7 @@
<?xml version="1.0" encoding="UTF-8"?>
<ui version="4.0">
- <class>Beautifier::Internal::Uncrustify::UncrustifyOptionsPage</class>
- <widget class="QWidget" name="Beautifier::Internal::Uncrustify::UncrustifyOptionsPage">
+ <class>Beautifier::Internal::UncrustifyOptionsPage</class>
+ <widget class="QWidget" name="Beautifier::Internal::UncrustifyOptionsPage">
<property name="geometry">
<rect>
<x>0</x>
diff --git a/src/plugins/beautifier/uncrustify/uncrustifysettings.cpp b/src/plugins/beautifier/uncrustify/uncrustifysettings.cpp
index d92c64a80d..6f94d5e49f 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifysettings.cpp
+++ b/src/plugins/beautifier/uncrustify/uncrustifysettings.cpp
@@ -41,9 +41,7 @@
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
-namespace {
const char USE_OTHER_FILES[] = "useOtherFiles";
const char USE_HOME_FILE[] = "useHomeFile";
const char USE_SPECIFIC_CONFIG_FILE_PATH[] = "useSpecificConfigFile";
@@ -51,10 +49,10 @@ const char SPECIFIC_CONFIG_FILE_PATH[] = "specificConfigFile";
const char USE_CUSTOM_STYLE[] = "useCustomStyle";
const char CUSTOM_STYLE[] = "customStyle";
const char FORMAT_ENTIRE_FILE_FALLBACK[] = "formatEntireFileFallback";
-}
+const char SETTINGS_NAME[] = "uncrustify";
UncrustifySettings::UncrustifySettings() :
- AbstractSettings(Constants::Uncrustify::SETTINGS_NAME, ".cfg")
+ AbstractSettings(SETTINGS_NAME, ".cfg")
{
connect(&m_versionProcess, QOverload<int, QProcess::ExitStatus>::of(&QProcess::finished),
this, &UncrustifySettings::parseVersionProcessResult);
@@ -144,7 +142,7 @@ QString UncrustifySettings::documentationFilePath() const
{
return Core::ICore::userResourcePath() + '/' + Beautifier::Constants::SETTINGS_DIRNAME + '/'
+ Beautifier::Constants::DOCUMENTATION_DIRNAME + '/'
- + Constants::Uncrustify::SETTINGS_NAME + ".xml";
+ + SETTINGS_NAME + ".xml";
}
void UncrustifySettings::createDocumentationFile() const
@@ -243,6 +241,5 @@ void UncrustifySettings::parseVersionProcessResult(int exitCode, QProcess::ExitS
parseVersion(QString::fromUtf8(m_versionProcess.readAllStandardError()), m_version);
}
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier
diff --git a/src/plugins/beautifier/uncrustify/uncrustifysettings.h b/src/plugins/beautifier/uncrustify/uncrustifysettings.h
index 07798ef1e1..23f9bd7ae7 100644
--- a/src/plugins/beautifier/uncrustify/uncrustifysettings.h
+++ b/src/plugins/beautifier/uncrustify/uncrustifysettings.h
@@ -31,7 +31,6 @@
namespace Beautifier {
namespace Internal {
-namespace Uncrustify {
class UncrustifySettings : public AbstractSettings
{
@@ -71,6 +70,5 @@ private:
void parseVersionProcessResult(int exitCode, QProcess::ExitStatus exitStatus);
};
-} // namespace Uncrustify
} // namespace Internal
} // namespace Beautifier