diff options
| author | Ahmad Samir <a.samirh78@gmail.com> | 2021-02-22 22:38:43 +0200 |
|---|---|---|
| committer | Ahmad Samir <a.samirh78@gmail.com> | 2021-03-06 01:33:45 +0200 |
| commit | ee35bdce8f6b08922b4c9e0c0c838e5f2c4a79ad (patch) | |
| tree | 591898b2b603d84329fe335675d054ad3e28bbd2 /src/gui/kconfiggui.cpp | |
| parent | 36f12b207f6a08f6d5fda7d53246a8abe70edf63 (diff) | |
| download | kconfig-ee35bdce8f6b08922b4c9e0c0c838e5f2c4a79ad.tar.gz kconfig-ee35bdce8f6b08922b4c9e0c0c838e5f2c4a79ad.tar.bz2 | |
Optimise string operations a bit
- Use QString::arg(Args...) instead of .arg().arg()
- Use QLatin1String for string comparisons, should be faster
- Use QLatin1String::arg() for better readability
- Add the comment dfaure suggested in the MR, to explain why it's '2%'
then '%1' in a QString().arg().arg()
NO_CHANGELOG
Diffstat (limited to 'src/gui/kconfiggui.cpp')
| -rw-r--r-- | src/gui/kconfiggui.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/gui/kconfiggui.cpp b/src/gui/kconfiggui.cpp index 938fbd93..cafe061d 100644 --- a/src/gui/kconfiggui.cpp +++ b/src/gui/kconfiggui.cpp @@ -13,7 +13,7 @@ static QString configName(const QString &id, const QString &key) { - return (QLatin1String("session/") + QGuiApplication::applicationName() + QLatin1Char('_') + id + QLatin1Char('_') + key); + return QLatin1String("session/%1_%2_%3").arg(QGuiApplication::applicationName(), id, key); } static KConfig *s_sessionConfig = nullptr; |