From 70cf2c76a560ab67058db4e785c73884f731930c Mon Sep 17 00:00:00 2001 From: Friedemann Kleint <Friedemann.Kleint@nokia.com> Date: Wed, 18 Mar 2009 17:05:48 +0100 Subject: [PATCH] Fixes: Use common default for lower casing file names --- src/libs/utils/classnamevalidatinglineedit.cpp | 2 +- src/plugins/cppeditor/cppclasswizard.cpp | 9 +++++---- src/plugins/cpptools/cppfilesettingspage.cpp | 3 ++- src/plugins/cpptools/cpptoolsconstants.h | 1 + src/plugins/designer/cpp/formclasswizard.cpp | 9 +++++---- src/plugins/designer/cpp/formclasswizardpage.cpp | 1 + 6 files changed, 15 insertions(+), 10 deletions(-) diff --git a/src/libs/utils/classnamevalidatinglineedit.cpp b/src/libs/utils/classnamevalidatinglineedit.cpp index 2697728a1aa..86f86a124d0 100644 --- a/src/libs/utils/classnamevalidatinglineedit.cpp +++ b/src/libs/utils/classnamevalidatinglineedit.cpp @@ -51,7 +51,7 @@ ClassNameValidatingLineEditPrivate:: ClassNameValidatingLineEditPrivate() : m_nameRegexp(QLatin1String("[a-zA-Z_][a-zA-Z0-9_]*(::[a-zA-Z_][a-zA-Z0-9_]*)*")), m_namespaceDelimiter(QLatin1String("::")), m_namespacesEnabled(false), - m_lowerCaseFileName(false) + m_lowerCaseFileName(true) { QTC_ASSERT(m_nameRegexp.isValid(), return); } diff --git a/src/plugins/cppeditor/cppclasswizard.cpp b/src/plugins/cppeditor/cppclasswizard.cpp index 134bbeb8222..d56875e213c 100644 --- a/src/plugins/cppeditor/cppclasswizard.cpp +++ b/src/plugins/cppeditor/cppclasswizard.cpp @@ -94,10 +94,11 @@ ClassNamePage::ClassNamePage(QWidget *parent) : // Retrieve settings of CppTools plugin. static inline bool lowerCaseFiles(const Core::ICore *core) { - QString camelCaseSettingsKey = QLatin1String(CppTools::Constants::CPPTOOLS_SETTINGSGROUP); - camelCaseSettingsKey += QLatin1Char('/'); - camelCaseSettingsKey += QLatin1String(CppTools::Constants::LOWERCASE_CPPFILES_KEY); - return core->settings()->value(camelCaseSettingsKey, QVariant(false)).toBool(); + QString lowerCaseSettingsKey = QLatin1String(CppTools::Constants::CPPTOOLS_SETTINGSGROUP); + lowerCaseSettingsKey += QLatin1Char('/'); + lowerCaseSettingsKey += QLatin1String(CppTools::Constants::LOWERCASE_CPPFILES_KEY); + const bool lowerCaseDefault = CppTools::Constants::lowerCaseFilesDefault; + return core->settings()->value(lowerCaseSettingsKey, QVariant(lowerCaseDefault)).toBool(); } // Set up new class widget from settings diff --git a/src/plugins/cpptools/cppfilesettingspage.cpp b/src/plugins/cpptools/cppfilesettingspage.cpp index a5bd11ac5dc..cdebf13a7c5 100644 --- a/src/plugins/cpptools/cppfilesettingspage.cpp +++ b/src/plugins/cpptools/cppfilesettingspage.cpp @@ -64,7 +64,8 @@ void CppFileSettings::fromSettings(QSettings *s) s->beginGroup(QLatin1String(Constants::CPPTOOLS_SETTINGSGROUP)); headerSuffix= s->value(QLatin1String(headerSuffixKeyC), QLatin1String("h")).toString(); sourceSuffix = s->value(QLatin1String(sourceSuffixKeyC), QLatin1String("cpp")).toString(); - lowerCaseFiles = s->value(QLatin1String(Constants::LOWERCASE_CPPFILES_KEY), QVariant(true)).toBool(); + const bool lowerCaseDefault = Constants::lowerCaseFilesDefault; + lowerCaseFiles = s->value(QLatin1String(Constants::LOWERCASE_CPPFILES_KEY), QVariant(lowerCaseDefault)).toBool(); s->endGroup(); } diff --git a/src/plugins/cpptools/cpptoolsconstants.h b/src/plugins/cpptools/cpptoolsconstants.h index ef04db2c44b..73002aaface 100644 --- a/src/plugins/cpptools/cpptoolsconstants.h +++ b/src/plugins/cpptools/cpptoolsconstants.h @@ -46,6 +46,7 @@ const char * const CPP_HEADER_MIMETYPE = "text/x-c++hdr"; // QSettings keys for use by the "New Class" wizards. const char * const CPPTOOLS_SETTINGSGROUP = "CppTools"; const char * const LOWERCASE_CPPFILES_KEY = "LowerCaseFiles"; +enum { lowerCaseFilesDefault = 1 }; const char * const CPP_SETTINGS_ID = QT_TRANSLATE_NOOP("CppTools", "File naming conventions"); const char * const CPP_SETTINGS_CATEGORY = QT_TRANSLATE_NOOP("CppTools", "C++"); diff --git a/src/plugins/designer/cpp/formclasswizard.cpp b/src/plugins/designer/cpp/formclasswizard.cpp index f6346b63d2e..d208372a724 100644 --- a/src/plugins/designer/cpp/formclasswizard.cpp +++ b/src/plugins/designer/cpp/formclasswizard.cpp @@ -67,10 +67,11 @@ QString FormClassWizard::formSuffix() const // Retrieve settings of CppTools plugin. static inline bool lowerCaseFiles() { - QString camelCaseSettingsKey = QLatin1String(CppTools::Constants::CPPTOOLS_SETTINGSGROUP); - camelCaseSettingsKey += QLatin1Char('/'); - camelCaseSettingsKey += QLatin1String(CppTools::Constants::LOWERCASE_CPPFILES_KEY); - return Core::ICore::instance()->settings()->value(camelCaseSettingsKey, QVariant(false)).toBool(); + QString lowerCaseSettingsKey = QLatin1String(CppTools::Constants::CPPTOOLS_SETTINGSGROUP); + lowerCaseSettingsKey += QLatin1Char('/'); + lowerCaseSettingsKey += QLatin1String(CppTools::Constants::LOWERCASE_CPPFILES_KEY); + const bool lowerCaseDefault = CppTools::Constants::lowerCaseFilesDefault; + return Core::ICore::instance()->settings()->value(lowerCaseSettingsKey, QVariant(lowerCaseDefault)).toBool(); } QWizard *FormClassWizard::createWizardDialog(QWidget *parent, diff --git a/src/plugins/designer/cpp/formclasswizardpage.cpp b/src/plugins/designer/cpp/formclasswizardpage.cpp index 0b7f4c24996..d469eb9edb7 100644 --- a/src/plugins/designer/cpp/formclasswizardpage.cpp +++ b/src/plugins/designer/cpp/formclasswizardpage.cpp @@ -84,6 +84,7 @@ static inline bool lowerCaseFiles(const Core::ICore *core) QString camelCaseSettingsKey = QLatin1String(CppTools::Constants::CPPTOOLS_SETTINGSGROUP); camelCaseSettingsKey += QLatin1Char('/'); camelCaseSettingsKey += QLatin1String(CppTools::Constants::LOWERCASE_CPPFILES_KEY); + return core->settings()->value(camelCaseSettingsKey, QVariant(false)).toBool(); } -- GitLab