X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fview%2Fsignal.cpp;h=59817eba2c334fcad882d296f5f0a4ac158e7d17;hp=3312c5924576b9517c5aa6425de4f935cbf95e63;hb=f4e57597347e47a4ea58fbdc7b0a22e07f1c0ede;hpb=4871ed92f2d9e6e514223383ba16e6ad78c81161

diff --git a/pv/view/signal.cpp b/pv/view/signal.cpp
index 3312c592..59817eba 100644
--- a/pv/view/signal.cpp
+++ b/pv/view/signal.cpp
@@ -21,32 +21,36 @@
 #include <extdef.h>
 
 #include <assert.h>
-#include <math.h>
+#include <cmath>
 
 #include <QApplication>
 #include <QFormLayout>
+#include <QKeyEvent>
+#include <QLineEdit>
 #include <QMenu>
 
-#include <libsigrok/libsigrok.h>
+#include <libsigrokcxx/libsigrokcxx.hpp>
 
-#include "signal.h"
-#include "view.h"
+#include "pv/data/signalbase.hpp"
 
-#include <pv/device/devinst.h>
+#include "signal.hpp"
+#include "view.hpp"
 
-using boost::shared_ptr;
+using std::shared_ptr;
+using std::make_shared;
 
 namespace pv {
-namespace view {
+namespace views {
+namespace TraceView {
 
-const char *const ProbeNames[] = {
+const char *const ChannelNames[] = {
 	"CLK",
 	"DATA",
 	"IN",
 	"OUT",
 	"RST",
-	"Tx",
-	"Rx",
+	"TX",
+	"RX",
 	"EN",
 	"SCLK",
 	"MOSI",
@@ -56,54 +60,81 @@ const char *const ProbeNames[] = {
 	"SCL"
 };
 
-Signal::Signal(shared_ptr<pv::device::DevInst> dev_inst,
-	const sr_channel *const probe) :
-	Trace(probe->name),
-	_dev_inst(dev_inst),
-	_probe(probe),
-	_name_widget(NULL),
-	_updating_name_widget(false)
+Signal::Signal(pv::Session &session,
+	std::shared_ptr<data::SignalBase> channel) :
+	Trace(channel),
+	session_(session),
+	scale_handle_(make_shared<SignalScaleHandle>(*this)),
+	items_({scale_handle_}),
+	name_widget_(nullptr)
 {
-	assert(_probe);
+	assert(base_);
+
+	connect(base_.get(), SIGNAL(enabled_changed(bool)),
+		this, SLOT(on_enabled_changed(bool)));
 }
 
 void Signal::set_name(QString name)
 {
 	Trace::set_name(name);
-	_updating_name_widget = true;
-	_name_widget->setEditText(name);
-	_updating_name_widget = false;
+
+	if (name != name_widget_->currentText())
+		name_widget_->setEditText(name);
 }
 
 bool Signal::enabled() const
 {
-	return _probe->enabled;
+	return base_->enabled();
+}
+
+shared_ptr<data::SignalBase> Signal::base() const
+{
+	return base_;
+}
+
+void Signal::save_settings(QSettings &settings) const
+{
+	(void)settings;
+}
+
+void Signal::restore_settings(QSettings &settings)
+{
+	(void)settings;
 }
 
-void Signal::enable(bool enable)
+const ViewItemOwner::item_list& Signal::child_items() const
 {
-	_dev_inst->enable_probe(_probe, enable);
-	visibility_changed();
+	return items_;
 }
 
-const sr_channel* Signal::probe() const
+void Signal::paint_back(QPainter &p, const ViewItemPaintParams &pp)
 {
-	return _probe;
+	if (base_->enabled())
+		Trace::paint_back(p, pp);
 }
 
 void Signal::populate_popup_form(QWidget *parent, QFormLayout *form)
 {
-	_name_widget = new QComboBox(parent);
-	_name_widget->setEditable(true);
+	name_widget_ = new QComboBox(parent);
+	name_widget_->setEditable(true);
+	name_widget_->setCompleter(nullptr);
 
-	for(unsigned int i = 0; i < countof(ProbeNames); i++)
-		_name_widget->insertItem(i, ProbeNames[i]);
-	_name_widget->setEditText(_probe->name);
+	for (unsigned int i = 0; i < countof(ChannelNames); i++)
+		name_widget_->insertItem(i, ChannelNames[i]);
 
-	connect(_name_widget, SIGNAL(editTextChanged(const QString&)),
-		this, SLOT(on_text_changed(const QString&)));
+	const int index = name_widget_->findText(base_->name(), Qt::MatchExactly);
 
-	form->addRow(tr("Name"), _name_widget);
+	if (index == -1) {
+		name_widget_->insertItem(0, base_->name());
+		name_widget_->setCurrentIndex(0);
+	} else {
+		name_widget_->setCurrentIndex(index);
+	}
+
+	connect(name_widget_, SIGNAL(editTextChanged(const QString&)),
+		this, SLOT(on_nameedit_changed(const QString&)));
+
+	form->addRow(tr("Name"), name_widget_);
 
 	add_colour_option(parent, form);
 }
@@ -127,10 +158,32 @@ void Signal::delete_pressed()
 	on_disable();
 }
 
+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);
+
+	Trace::on_name_changed(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