From: Soeren Apel <redacted>
Date: Fri, 29 Nov 2019 22:36:56 +0000 (+0100)
Subject: DecoderOutput: Add format selector
X-Git-Url: http://sigrok.org/gitweb/?a=commitdiff_plain;h=861ab3a46dd4691cf9fd519bdf2fce7edbee0971;hp=bdbc561f1ac395449f1b3afa7d2d0538dd1aa400;p=pulseview.git

DecoderOutput: Add format selector
---

diff --git a/pv/views/decoder_output/view.cpp b/pv/views/decoder_output/view.cpp
index d4077476..fdf4bc7d 100644
--- a/pv/views/decoder_output/view.cpp
+++ b/pv/views/decoder_output/view.cpp
@@ -43,7 +43,8 @@ View::View(Session &session, bool is_main_view, QMainWindow *parent) :
 	ViewBase(session, is_main_view, parent),
 
 	// Note: Place defaults in View::reset_view_state(), not here
-	signal_selector_(new QComboBox())
+	signal_selector_(new QComboBox()),
+	format_selector_(new QComboBox())
 {
 	QVBoxLayout *root_layout = new QVBoxLayout(this);
 	root_layout->setContentsMargins(0, 0, 0, 0);
@@ -56,6 +57,12 @@ View::View(Session &session, bool is_main_view, QMainWindow *parent) :
 	// Populate toolbar
 	toolbar->addWidget(new QLabel(tr("Decoder:")));
 	toolbar->addWidget(signal_selector_);
+	toolbar->addSeparator();
+	toolbar->addWidget(new QLabel(tr("Show data as")));
+	toolbar->addWidget(format_selector_);
+
+	// Add format types
+	format_selector_->addItem(tr("Hexdump"), qVariantFromValue(QString("text/hexdump")));
 
 	reset_view_state();
 }
@@ -82,6 +89,7 @@ void View::clear_signals()
 void View::clear_decode_signals()
 {
 	signal_selector_->clear();
+	format_selector_->setCurrentIndex(0);
 }
 
 void View::add_decode_signal(shared_ptr<data::DecodeSignal> signal)
@@ -89,12 +97,12 @@ void View::add_decode_signal(shared_ptr<data::DecodeSignal> signal)
 	connect(signal.get(), SIGNAL(name_changed(const QString&)),
 		this, SLOT(on_signal_name_changed(const QString&)));
 
-	signal_selector_->addItem(signal->name(), qVariantFromValue(signal.get()));
+	signal_selector_->addItem(signal->name(), QVariant::fromValue(signal.get()));
 }
 
 void View::remove_decode_signal(shared_ptr<data::DecodeSignal> signal)
 {
-	int index = signal_selector_->findData(qVariantFromValue(signal.get()));
+	int index = signal_selector_->findData(QVariant::fromValue(signal.get()));
 
 	if (index != -1)
 		signal_selector_->removeItem(index);
@@ -117,7 +125,7 @@ void View::on_signal_name_changed(const QString &name)
 	SignalBase *sb = qobject_cast<SignalBase*>(QObject::sender());
 	assert(sb);
 
-	int index = signal_selector_->findData(qVariantFromValue(sb));
+	int index = signal_selector_->findData(QVariant::fromValue(sb));
 	if (index != -1)
 		signal_selector_->setItemText(index, name);
 }
diff --git a/pv/views/decoder_output/view.hpp b/pv/views/decoder_output/view.hpp
index 4f772c82..cd90b5ca 100644
--- a/pv/views/decoder_output/view.hpp
+++ b/pv/views/decoder_output/view.hpp
@@ -62,7 +62,7 @@ private Q_SLOTS:
 	void on_signal_name_changed(const QString &name);
 
 private:
-	QComboBox *signal_selector_;
+	QComboBox *signal_selector_, *format_selector_;
 };
 
 } // namespace decoder_output