Change namespace for the trace view and implement ViewBase

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

diff --git a/pv/view/signal.cpp b/pv/view/signal.cpp
index a8de4d504aaa36b0fdb9960fb4f5b72c5b2391ae..59817eba2c334fcad882d296f5f0a4ac158e7d17 100644 (file)
--- a/pv/view/signal.cpp
+++ b/pv/view/signal.cpp
@@ -40,7 +40,8 @@ using std::shared_ptr;
 using std::make_shared;
 
 namespace pv {
-namespace view {
+namespace views {
+namespace TraceView {
 
 const char *const ChannelNames[] = {
        "CLK",
@@ -68,6 +69,9 @@ Signal::Signal(pv::Session &session,
        name_widget_(nullptr)
 {
        assert(base_);
+
+       connect(base_.get(), SIGNAL(enabled_changed(bool)),
+               this, SLOT(on_enabled_changed(bool)));
 }
 
 void Signal::set_name(QString name)
@@ -83,17 +87,19 @@ bool Signal::enabled() const
        return base_->enabled();
 }
 
-void Signal::enable(bool enable)
+shared_ptr<data::SignalBase> Signal::base() const
 {
-       base_->set_enabled(enable);
+       return base_;
+}
 
-       if (owner_)
-               owner_->extents_changed(true, true);
+void Signal::save_settings(QSettings &settings) const
+{
+       (void)settings;
 }
 
-shared_ptr<data::SignalBase> Signal::base() const
+void Signal::restore_settings(QSettings &settings)
 {
-       return base_;
+       (void)settings;
 }
 
 const ViewItemOwner::item_list& Signal::child_items() const
@@ -154,6 +160,11 @@ void Signal::delete_pressed()
 
 void Signal::on_name_changed(const QString &text)
 {
+       // On startup, this event is fired when a session restores signal
+       // names. However, the name widget hasn't yet been created.
+       if (!name_widget_)
+               return;
+
        if (text != name_widget_->currentText())
                name_widget_->setEditText(text);
 
@@ -162,8 +173,17 @@ void Signal::on_name_changed(const QString &text)
 
 void Signal::on_disable()
 {
-       enable(false);
+       base_->set_enabled(false);
+}
+
+void Signal::on_enabled_changed(bool enabled)
+{
+       (void)enabled;
+
+       if (owner_)
+               owner_->extents_changed(true, true);
 }
 
-} // namespace view
+} // namespace TraceView
+} // namespace views
 } // namespace pv