From eb55ffde16f0f3c1df4c97c5229165acfda5637d Mon Sep 17 00:00:00 2001 From: Berthold Stoeger Date: Sun, 22 Mar 2020 16:43:21 +0100 Subject: cleanup: split uiLanguage() in two functions The uiLanguage() function was used for two purposes: to initialize the language related preferences and to read the current language. To make things more easy to follow, split this function in two: one for initializing, one for getting the current language. Moreover, don't return the current locale in an out-parameter as there is already a function to do that [getLocale()]. Signed-off-by: Berthold Stoeger --- desktop-widgets/preferences/preferences_language.cpp | 2 +- desktop-widgets/usermanual.cpp | 2 +- desktop-widgets/usersurvey.cpp | 4 ++-- 3 files changed, 4 insertions(+), 4 deletions(-) (limited to 'desktop-widgets') diff --git a/desktop-widgets/preferences/preferences_language.cpp b/desktop-widgets/preferences/preferences_language.cpp index 9de6efb8f..22b274a76 100644 --- a/desktop-widgets/preferences/preferences_language.cpp +++ b/desktop-widgets/preferences/preferences_language.cpp @@ -89,7 +89,7 @@ void PreferencesLanguage::syncSettings() qPrefLanguage::set_time_format(ui->timeFormatEntry->currentText()); qPrefLanguage::set_date_format(ui->dateFormatEntry->currentText()); qPrefLanguage::set_date_format_short(ui->shortDateFormatEntry->text()); - uiLanguage(NULL); + initUiLanguage(); QString qDateTimeWeb = tr("These will be used as is. This might not be what you intended.\nSee http://doc.qt.io/qt-5/qdatetime.html#toString"); QRegExp tfillegalchars("[^hHmszaApPt\\s:;\\.,]"); diff --git a/desktop-widgets/usermanual.cpp b/desktop-widgets/usermanual.cpp index 69e86864b..42ca4e0a7 100644 --- a/desktop-widgets/usermanual.cpp +++ b/desktop-widgets/usermanual.cpp @@ -64,7 +64,7 @@ UserManual::UserManual(QWidget *parent) : QDialog(parent) QString searchPath = getSubsurfaceDataPath("Documentation"); if (searchPath.size()) { // look for localized versions of the manual first - QString lang = uiLanguage(NULL); + QString lang = getUiLanguage(); QString prefix = searchPath.append("/user-manual"); QFile manual(prefix + "_" + lang + ".html"); if (!manual.exists()) diff --git a/desktop-widgets/usersurvey.cpp b/desktop-widgets/usersurvey.cpp index 326a51f9e..9f237abea 100644 --- a/desktop-widgets/usersurvey.cpp +++ b/desktop-widgets/usersurvey.cpp @@ -31,7 +31,7 @@ UserSurvey::UserSurvey(QWidget *parent) : QDialog(parent), QString osArch = SubsurfaceSysInfo::currentCpuArchitecture(); os.append(QString("&osCpuArch=%1").arg(osArch)); } - os.append(QString("&uiLang=%1").arg(uiLanguage(NULL))); + os.append(QString("&uiLang=%1").arg(getUiLanguage())); os.append(QString("&uuid=%1").arg(getUUID())); ui->system->setPlainText(getVersion()); } @@ -46,7 +46,7 @@ QString UserSurvey::getVersion() sysInfo.append(tr("\nCPU architecture: %1").arg(arch)); if (arch == "i386") sysInfo.append(tr("\nOS CPU architecture: %1").arg(SubsurfaceSysInfo::currentCpuArchitecture())); - sysInfo.append(tr("\nLanguage: %1").arg(uiLanguage(NULL))); + sysInfo.append(tr("\nLanguage: %1").arg(getUiLanguage())); return sysInfo; } -- cgit v1.2.3-70-g09d2