Commit 7b0149c7 authored by Kevin Ottens's avatar Kevin Ottens
Browse files

Fix reset to defaults in the fonts KCM

Summary:
Turns out one of the properties was utterly broken emitting the wrong
signal, and the defaults() method was missing a piece of state to reset.

Since it was making my eyes bleed I also removed the "const int &"
parameters in the process. :-)

Reviewers: #plasma, bport, davidedmundson, mart

Subscribers: plasma-devel

Tags: #plasma

Differential Revision: https://phabricator.kde.org/D25094
parent 69d649e5
......@@ -316,6 +316,7 @@ bool FontAASettings::save(KXftConfig::AntiAliasing::State aaState)
void FontAASettings::defaults()
{
setExclude(false);
setExcludeTo(15);
setExcludeFrom(8);
setAntiAliasing(true);
......@@ -345,7 +346,7 @@ bool FontAASettings::exclude() const
return m_state.exclude;
}
void FontAASettings::setExcludeTo(const int &excludeTo)
void FontAASettings::setExcludeTo(int excludeTo)
{
if (m_state.excludeTo == excludeTo) {
return;
......@@ -360,14 +361,14 @@ int FontAASettings::excludeTo() const
return m_state.excludeTo;
}
void FontAASettings::setExcludeFrom(const int &excludeTo)
void FontAASettings::setExcludeFrom(int excludeFrom)
{
if (m_state.excludeFrom == excludeTo) {
if (m_state.excludeFrom == excludeFrom) {
return;
}
m_state.excludeFrom = excludeTo;
emit excludeToChanged();
m_state.excludeFrom = excludeFrom;
emit excludeFromChanged();
}
int FontAASettings::excludeFrom() const
......
......@@ -84,10 +84,10 @@ public:
void setExclude(bool exclude);
bool exclude() const;
void setExcludeTo(const int &excludeTo);
void setExcludeTo(int excludeTo);
int excludeTo() const;
void setExcludeFrom(const int &excludeTo);
void setExcludeFrom(int excludeFrom);
int excludeFrom() const;
void setAntiAliasing(bool antiAliasing);
......
Markdown is supported
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment