Initialise Trace widgets when the trace is received by the View

[pulseview.git] / pv / view / logicsignal.cpp

diff --git a/pv/view/logicsignal.cpp b/pv/view/logicsignal.cpp
index ea138fa76d2b2285e50c588ed26b833c4e1df170..0c934cc72cb0f3de8a675332de3115deb86b9ee4 100644 (file)
--- a/pv/view/logicsignal.cpp
+++ b/pv/view/logicsignal.cpp
@@ -58,45 +58,63 @@ LogicSignal::LogicSignal(pv::SigSession &session, const sr_probe *const probe,
        shared_ptr<data::Logic> data) :
        Signal(session, probe),
        _data(data),
-       _separator(this),
-       _icon_trigger_none(":/icons/trigger-none.svg"),
-       _trigger_none(_icon_trigger_none, tr("No trigger"), this),
-       _icon_trigger_rising(":/icons/trigger-rising.svg"),
-       _trigger_rising(_icon_trigger_rising,
-               tr("Trigger on rising edge"), this),
-       _icon_trigger_high(":/icons/trigger-high.svg"),
-       _trigger_high(_icon_trigger_high,
-               tr("Trigger on high level"), this),
-       _icon_trigger_falling(":/icons/trigger-falling.svg"),
-       _trigger_falling(_icon_trigger_falling,
-               tr("Trigger on falling edge"), this),
-       _icon_trigger_low(":/icons/trigger-low.svg"),
-       _trigger_low(_icon_trigger_low,
-               tr("Trigger on low level"), this),
-       _icon_trigger_change(":/icons/trigger-change.svg"),
-       _trigger_change(_icon_trigger_change,
-               tr("Trigger on rising or falling edge"), this)
+       _separator(NULL),
+       _trigger_none(NULL),
+       _trigger_rising(NULL),
+       _trigger_high(NULL),
+       _trigger_falling(NULL),
+       _trigger_low(NULL),
+       _trigger_change(NULL)
 {
        _colour = SignalColours[probe->index % countof(SignalColours)];
+}
+
+LogicSignal::~LogicSignal()
+{
+}
 
-       _separator.setSeparator(true);
+void LogicSignal::init_context_bar_actions(QWidget *parent)
+{
+       Signal::init_context_bar_actions(parent);
+
+       _separator = new QAction(parent);
+       _separator->setSeparator(true);
 
-       connect(&_trigger_none, SIGNAL(triggered()),
+       _trigger_none = new QAction(QIcon(":/icons/trigger-none.svg"),
+               tr("No trigger"), this);
+       _trigger_none->setCheckable(true);
+       connect(_trigger_none, SIGNAL(triggered()),
                this, SLOT(on_trigger_none()));
-       connect(&_trigger_rising, SIGNAL(triggered()),
+
+       _trigger_rising = new QAction(QIcon(":/icons/trigger-rising.svg"),
+               tr("Trigger on rising edge"), this);
+       _trigger_rising->setCheckable(true);
+       connect(_trigger_rising, SIGNAL(triggered()),
                this, SLOT(on_trigger_rising()));
-       connect(&_trigger_high, SIGNAL(triggered()),
+
+       _trigger_high = new QAction(QIcon(":/icons/trigger-low.svg"),
+               tr("Trigger on high level"), this);
+       _trigger_high->setCheckable(true);
+       connect(_trigger_high, SIGNAL(triggered()),
                this, SLOT(on_trigger_high()));
-       connect(&_trigger_falling, SIGNAL(triggered()),
+
+       _trigger_falling = new QAction(QIcon(":/icons/trigger-falling.svg"),
+               tr("Trigger on falling edge"), this);
+       _trigger_falling->setCheckable(true);
+       connect(_trigger_falling, SIGNAL(triggered()),
                this, SLOT(on_trigger_falling()));
-       connect(&_trigger_low, SIGNAL(triggered()),
+
+       _trigger_low = new QAction(QIcon(":/icons/trigger-low.svg"),
+               tr("Trigger on low level"), this);
+       _trigger_low->setCheckable(true);
+       connect(_trigger_low, SIGNAL(triggered()),
                this, SLOT(on_trigger_low()));
-       connect(&_trigger_change, SIGNAL(triggered()),
-               this, SLOT(on_trigger_change()));
-}
 
-LogicSignal::~LogicSignal()
-{
+       _trigger_change = new QAction(QIcon(":/icons/trigger-change.svg"),
+               tr("Trigger on rising or falling edge"), this);
+       _trigger_change->setCheckable(true);
+       connect(_trigger_change, SIGNAL(triggered()),
+               this, SLOT(on_trigger_change()));
 }
 
 const list<QAction*> LogicSignal::get_context_bar_actions()
@@ -104,7 +122,7 @@ const list<QAction*> LogicSignal::get_context_bar_actions()
        GVariant *gvar;
        list<QAction*> actions;
 
-       actions.push_back(&_name_action);
+       actions.push_back(_name_action);
 
        // Add the trigger actions
        const sr_dev_inst *const sdi = _session.get_device();
@@ -114,20 +132,22 @@ const list<QAction*> LogicSignal::get_context_bar_actions()
                        g_variant_get_string(gvar, NULL);
 
                if (trig_types && trig_types[0] != '\0') {
-                       actions.push_back(&_separator);
+                       actions.push_back(_separator);
 
-                       actions.push_back(&_trigger_none);
+                       actions.push_back(_trigger_none);
 
                        add_trigger_action(trig_types, 'r',
-                               &_trigger_rising, actions);
+                               _trigger_rising, actions);
                        add_trigger_action(trig_types, '1',
-                               &_trigger_high, actions);
+                               _trigger_high, actions);
                        add_trigger_action(trig_types, 'f',
-                               &_trigger_falling, actions);
+                               _trigger_falling, actions);
                        add_trigger_action(trig_types, '0',
-                               &_trigger_low, actions);
+                               _trigger_low, actions);
                        add_trigger_action(trig_types, 'c',
-                               &_trigger_change, actions);
+                               _trigger_change, actions);
+               
+                       update_trigger_actions();
                }
 
                g_variant_unref(gvar);
@@ -246,6 +266,18 @@ void LogicSignal::add_trigger_action(const char *trig_types, char type,
                }
 }
 
+void LogicSignal::update_trigger_actions()
+{
+       const char cur_trigger = _probe->trigger ?
+               _probe->trigger[0] : '\0';
+       _trigger_none->setChecked(cur_trigger == '\0');
+       _trigger_rising->setChecked(cur_trigger == 'r');
+       _trigger_high->setChecked(cur_trigger == '1');
+       _trigger_falling->setChecked(cur_trigger == 'f');
+       _trigger_low->setChecked(cur_trigger == '0');
+       _trigger_change->setChecked(cur_trigger == 'c');
+}
+
 void LogicSignal::set_trigger(char type)
 {
        const char trigger_type_string[2] = {type, 0};
@@ -262,6 +294,8 @@ void LogicSignal::set_trigger(char type)
                sr_dev_trigger_set(sdi, i, (i == _probe->index) ?
                        trigger_string : NULL);
        }
+
+       update_trigger_actions();
 }
 
 void LogicSignal::on_trigger_none()