on xenial help sign in
Removed *changed() notifications for Filter properties and marked them CONSTANT.
commit0431d6004aaff9c97d414804bac826dc8e453b82 authorPawel Stolowski <pawel.stolowski@canonical.com> dateWed 14 Aug 2013, 16:15 (13 years ago) parent995ff08
2 files changed+7−59
plugins/Unity/filter.cpp−33 plugins/Unity/filter.h+7−26
plugins/Unity/filter.cpp−33
@@ -99,20 +99,7 @@ void Filter::clear()
9999
100100void Filter::setUnityFilter(unity::dash::Filter::Ptr unityFilter)
101101{
102− if (m_unityFilter != nullptr) {
103− m_signals.disconnectAll();
104− }
105−
106102 m_unityFilter = unityFilter;
107−
108− /* Property change signals */
109− m_signals << m_unityFilter->id.changed.connect(sigc::mem_fun(this, &Filter::onIdChanged))
110− << m_unityFilter->name.changed.connect(sigc::mem_fun(this, &Filter::onNameChanged))
111− << m_unityFilter->icon_hint.changed.connect(sigc::mem_fun(this, &Filter::onIconHintChanged))
112− << m_unityFilter->renderer_name.changed.connect(sigc::mem_fun(this, &Filter::onRendererNameChanged))
113− << m_unityFilter->visible.changed.connect(sigc::mem_fun(this, &Filter::visibleChanged))
114− << m_unityFilter->collapsed.changed.connect(sigc::mem_fun(this, &Filter::collapsedChanged))
115− << m_unityFilter->filtering.changed.connect(sigc::mem_fun(this, &Filter::filteringChanged));
116103}
117104
118105Filter* Filter::newFromUnityFilter(unity::dash::Filter::Ptr unityFilter)
@@ -140,23 +127,3 @@ bool Filter::hasUnityFilter(unity::dash::Filter::Ptr unityFilter) const
140127{
141128 return m_unityFilter == unityFilter;
142129}
143−
144−void Filter::onIdChanged(std::string id)
145−{
146− Q_EMIT idChanged(QString::fromStdString(id));
147−}
148−
149−void Filter::onNameChanged(std::string name)
150−{
151− Q_EMIT nameChanged(QString::fromStdString(name));
152−}
153−
154−void Filter::onIconHintChanged(std::string iconHint)
155−{
156− Q_EMIT iconHintChanged(QString::fromStdString(iconHint));
157−}
158−
159−void Filter::onRendererNameChanged(std::string renderer)
160−{
161− Q_EMIT rendererNameChanged(QString::fromStdString(renderer));
162−}
plugins/Unity/filter.h+7−26
@@ -28,22 +28,19 @@
2828// libunity-core
2929#include <UnityCore/Filter.h>
3030
31−// local
32−#include "signalslist.h"
33−
3431class GenericOptionsModel;
3532
3633class Q_DECL_EXPORT Filter : public QObject
3734{
3835 Q_OBJECT
3936
40− Q_PROPERTY(QString id READ id NOTIFY idChanged)
41− Q_PROPERTY(QString name READ name NOTIFY nameChanged)
42− Q_PROPERTY(QString iconHint READ iconHint NOTIFY iconHintChanged)
43− Q_PROPERTY(QString rendererName READ rendererName NOTIFY rendererNameChanged)
44− Q_PROPERTY(bool visible READ visible NOTIFY visibleChanged)
45− Q_PROPERTY(bool collapsed READ collapsed NOTIFY collapsedChanged)
46− Q_PROPERTY(bool filtering READ filtering NOTIFY filteringChanged)
37+ Q_PROPERTY(QString id READ id CONSTANT)
38+ Q_PROPERTY(QString name READ name CONSTANT)
39+ Q_PROPERTY(QString iconHint READ iconHint CONSTANT)
40+ Q_PROPERTY(QString rendererName READ rendererName CONSTANT)
41+ Q_PROPERTY(bool visible READ visible CONSTANT)
42+ Q_PROPERTY(bool collapsed READ collapsed CONSTANT)
43+ Q_PROPERTY(bool filtering READ filtering CONSTANT)
4744 Q_PROPERTY(GenericOptionsModel* options READ options NOTIFY optionsChanged)
4845
4946public:
@@ -65,27 +62,11 @@ public:
6562 bool hasUnityFilter(unity::dash::Filter::Ptr unityFilter) const;
6663
6764Q_SIGNALS:
68− void idChanged(const QString &id);
69− void nameChanged(const QString &name);
70− void iconHintChanged(const QString &iconHint);
71− void rendererNameChanged(const QString &renderer);
72− void visibleChanged(bool);
73− void collapsedChanged(bool);
74− void filteringChanged(bool);
7565 void optionsChanged();
7666
7767protected:
7868 unity::dash::Filter::Ptr m_unityFilter;
7969 virtual void setUnityFilter(unity::dash::Filter::Ptr unityFilter);
80−
81−private:
82− void onIdChanged(std::string);
83− void onNameChanged(std::string);
84− void onIconHintChanged(std::string);
85− void onRendererNameChanged(std::string);
86−
87−private:
88− SignalsList m_signals;
8970};
9071
9172Q_DECLARE_METATYPE(Filter*)