Differential D10758

Use QLocale instead of QLocale::system
AbandonedPublic
Actions

Authored by aacid on Feb 23 2018, 12:36 AM.

Details

Reviewers
apol
ilic
Summary

That makes it work if someone called QLocale::setDefault like kxmlgui does when setting an individual app language

Test Plan

See https://bugs.kde.org/show_bug.cgi?id=390623

Diff Detail

Repository
R249 KI18n
Branch
master
Lint
No Linters Available
Unit
No Unit Test Coverage
aacid created this revision.Feb 23 2018, 12:36 AM
Restricted Application added a project: Frameworks. · View Herald TranscriptFeb 23 2018, 12:36 AM
Restricted Application added a subscriber: Frameworks. · View Herald Transcript
aacid requested review of this revision.Feb 23 2018, 12:36 AM
lueck added a subscriber: lueck.Feb 23 2018, 6:27 AM

using D10757, D10758, and D10759 the bug reported in https://bugs.kde.org/show_bug.cgi?id=390623 is fixed for me, thanks a lot

apol accepted this revision.Feb 23 2018, 11:35 AM
This revision is now accepted and ready to land.Feb 23 2018, 11:35 AM
ilic accepted this revision.Feb 23 2018, 10:11 PM
aacid added a comment.Feb 23 2018, 11:48 PM

there's a new revision of D10757 that shouldn't need this at all.

aacid abandoned this revision.Feb 25 2018, 10:46 AM

Revision Contents
Changeset List

PathPackages
Msrc/klocalizedstring.cpp (2 lines)
Msrc/main.cpp (2 lines)

Diff 27827

View Options

src/klocalizedstring.cpp

Loading...
View Options

src/main.cpp

Loading...