| Age | Commit message (Collapse) | Author |
|---|
|
Before, these kind of read-only operations did a lot of allocations:
1) allocate a list of all sub groups
2) for the above, also allocate a sub-group match key
3) iterate over sub groups, allocate a list of all keys in there
and then finally check whether that list is non-empty
All of the above is now done without a single allocation, by simply
iterating over the list of entries.
Note: The whole list was iterated even before in allSubGroups. Now
we still do that, but check for non-empty keys in the group or
sub group directly. Much more efficient.
Note2: While at it, allSubGroups is also optimized to not require the
allocation of the subgroup match key.
REVIEW: 118586
forward-port of commit eaffd50adfd7fcbeafadb0248904176808b4499d
|
|
forward-port of commit 416559cdb86f9cacf4ea4ed7530cbbbdb427302f
|
|
The underlying KConfig used QLocale::name() for getting the locale
aware part. But this returns "de_DE" while the desktop files store
"de".
In addition it constructs a QLocale object instead of using the
system locale. This has the advantage that the usage of
QLocale::setDafault() gets honored by KConfig.
REVIEW: 118564
|
|
- use Q_NULLPTR instead of 0 or NULL
- simplify some foreach loops
- use QStringLiteral in a few places
- added a few consts
REVIEW: 118666
|
|
This reverts commit 625e124898afad04ab5c3939b1b129b6014046cd.
Let's take the time to think about a better solution, possibly.
|
|
Only KSharedConfig::openConfig() and KConfig default constructor are affected.
KConfig("kdeglobals"), for instance, points to ~/.config so that it can be shared.
REVIEW: 117989
|
|
|
|
It is not on $PATH, so we cannot just expect QProcess to find it.
REVIEW: 117023
|
|
All this ifdef'd-out code just makes it hard to see what's going on.
REVIEW: 116962
|
|
|
|
|
|
|
|
|
|
|
|
Use git blame -w 867e7a5 to show authorship as it was before this commit.
|
|
|
