X-Git-Url: https://sigrok.org/gitweb/?p=pulseview.git;a=blobdiff_plain;f=pv%2Fsession.cpp;h=4dcc2e8b7b77f5da1cfce26781e694d16f23eb7f;hp=b022bd02c0c02681bd576a06fff085b89f27e0c8;hb=2601ce965fc455d23f5a964b84ddd74eb93705c4;hpb=f5a5b019c5679dbe6b4bc6f956c3820f47869bc1 diff --git a/pv/session.cpp b/pv/session.cpp index b022bd02..4dcc2e8b 100644 --- a/pv/session.cpp +++ b/pv/session.cpp @@ -91,6 +91,9 @@ using sigrok::Session; using Glib::VariantBase; namespace pv { + +shared_ptr Session::sr_context; + Session::Session(DeviceManager &device_manager, QString name) : device_manager_(device_manager), default_name_(name), @@ -316,9 +319,9 @@ void Session::restore_settings(QSettings &settings) int decode_signals = settings.value("decode_signals").toInt(); for (int i = 0; i < decode_signals; i++) { - settings.beginGroup("decode_signal" + QString::number(i++)); - // TODO Split up add_decoder() into add_decode_signal() and add_decoder(), - // then call add_decode_signal() and signal->restore_settings() here + settings.beginGroup("decode_signal" + QString::number(i)); + shared_ptr signal = add_decode_signal(); + signal->restore_settings(settings); settings.endGroup(); } #endif @@ -619,8 +622,6 @@ void Session::register_view(shared_ptr view) switch (signalbase->type()) { case data::SignalBase::AnalogChannel: case data::SignalBase::LogicChannel: - case data::SignalBase::A2LChannel: - break; case data::SignalBase::DecodeChannel: #ifdef ENABLE_DECODE trace_view->add_decode_signal( @@ -682,31 +683,27 @@ const unordered_set< shared_ptr > Session::signalbases() const } #ifdef ENABLE_DECODE -bool Session::add_decoder(srd_decoder *const dec) +shared_ptr Session::add_decode_signal() { - if (!dec) - return false; + shared_ptr signal; try { // Create the decode signal - shared_ptr signal = - make_shared(*this); + signal = make_shared(*this); signalbases_.insert(signal); // Add the decode signal to all views for (shared_ptr view : views_) view->add_decode_signal(signal); - - // Add decoder - signal->stack_decoder(dec); } catch (runtime_error e) { - return false; + remove_decode_signal(signal); + return nullptr; } signals_changed(); - return true; + return signal; } void Session::remove_decode_signal(shared_ptr signal)