[Konsole-devel] [konsole/Applications/15.12] src: Call KLocalizedString::setApplicationDomain("konsole");

Albert Astals Cid aacid at kde.org
Sun Nov 29 18:41:07 UTC 2015

Git commit 740675243c0e573dd25fc65ae329b08be4538d58 by Albert Astals Cid.
Committed on 29/11/2015 at 18:39.
Pushed by aacid into branch 'Applications/15.12'.

Call KLocalizedString::setApplicationDomain("konsole");

konsole is using
that is wrong for applications, apps need to use
since that is needed for some things like the transltators tab in the about

I have not removed -DTRANSLATION_DOMAIN=\"konsole\" because on the other hand
this code is also a library (i.e. the konsole kpart).

The "being pedanticly correct" fix would be using -DTRANSLATION_DOMAIN=\"konsole\"
only to compile the files that are part of the kpart but that's prone to
break at some point and using both KLocalizedString::setApplicationDomain and
-DTRANSLATION_DOMAIN does not create any problem so i went that route

CCMAIL: konsole-devel at kde.org

M  +2    -0    src/main.cpp


diff --git a/src/main.cpp b/src/main.cpp
index 7c5d953..5e1420e 100644
--- a/src/main.cpp
+++ b/src/main.cpp
@@ -81,6 +81,8 @@ extern "C" int KDE_EXPORT kdemain(int argc, char** argv)
+    KLocalizedString::setApplicationDomain("konsole");
     KAboutData about(QStringLiteral("konsole"),
                      i18nc("@title", "Konsole"),

More information about the konsole-devel mailing list