X-Git-Url: http://git.code-monkey.de/?a=blobdiff_plain;f=pv%2Fview%2Fview.cpp;h=ba095fb808d083a98397740554de5bc3a4096c0e;hb=3e769a374963f37f7d52b65ef6c56bcdf1ef11fe;hp=1d55e14f7c91f20f55d09bffc1d4540d8819ddb6;hpb=4b4f1c0dc17b481b798e60bbc3fa639c09a9c788;p=pulseview.git diff --git a/pv/view/view.cpp b/pv/view/view.cpp index 1d55e14..ba095fb 100644 --- a/pv/view/view.cpp +++ b/pv/view/view.cpp @@ -22,9 +22,10 @@ #include #endif -#include -#include -#include +#include +#include +#include +#include #include #include @@ -42,9 +43,13 @@ #include "pv/data/logic.h" #include "pv/data/logicsnapshot.h" +using boost::shared_lock; +using boost::shared_mutex; using pv::data::SignalData; +using std::back_inserter; using std::deque; using std::list; +using std::lock_guard; using std::max; using std::make_pair; using std::min; @@ -62,10 +67,6 @@ const double View::MinScale = 1e-15; const int View::MaxScrollValue = INT_MAX / 2; -const int View::SignalHeight = 30; -const int View::SignalMargin = 10; -const int View::SignalSnapGridSize = 10; - const QColor View::CursorAreaColour(220, 231, 243); const QSizeF View::LabelPadding(4, 0); @@ -92,6 +93,8 @@ View::View(SigSession &session, QWidget *parent) : connect(&_session, SIGNAL(signals_changed()), this, SLOT(signals_changed())); + connect(&_session, SIGNAL(capture_state_changed(int)), + this, SLOT(data_updated())); connect(&_session, SIGNAL(data_received()), this, SLOT(data_updated())); connect(&_session, SIGNAL(frame_ended()), @@ -102,8 +105,6 @@ View::View(SigSession &session, QWidget *parent) : connect(_cursors.second().get(), SIGNAL(time_changed()), this, SLOT(marker_time_changed())); - connect(_header, SIGNAL(geometry_updated()), - this, SLOT(on_geometry_updated())); connect(_header, SIGNAL(signals_moved()), this, SLOT(on_signals_moved())); @@ -117,6 +118,13 @@ View::View(SigSession &session, QWidget *parent) : connect(_cursorheader, SIGNAL(selection_changed()), this, SIGNAL(selection_changed())); + connect(this, SIGNAL(hover_point_changed()), + this, SLOT(on_hover_point_changed())); + + connect(&_lazy_event_handler, SIGNAL(timeout()), + this, SLOT(process_sticky_events())); + _lazy_event_handler.setSingleShot(true); + setViewport(_viewport); _viewport->installEventFilter(this); @@ -143,6 +151,26 @@ const SigSession& View::session() const return _session; } +View* View::view() +{ + return this; +} + +const View* View::view() const +{ + return this; +} + +Viewport* View::viewport() +{ + return _viewport; +} + +const Viewport* View::viewport() const +{ + return _viewport; +} + double View::scale() const { return _scale; @@ -153,9 +181,14 @@ double View::offset() const return _offset; } -int View::v_offset() const +int View::owner_v_offset() const +{ + return -_v_offset; +} + +unsigned int View::depth() const { - return _v_offset; + return 0; } void View::zoom(double steps) @@ -188,9 +221,6 @@ void View::zoom_one_to_one() { using pv::data::SignalData; - const vector< shared_ptr > sigs( - session().get_signals()); - // Make a set of all the visible data objects set< shared_ptr > visible_data = get_visible_data(); if (visible_data.empty()) @@ -225,54 +255,10 @@ void View::set_scale_offset(double scale, double offset) scale_offset_changed(); } -vector< shared_ptr > View::get_traces() const -{ - const vector< shared_ptr > sigs( - session().get_signals()); -#ifdef ENABLE_DECODE - const vector< shared_ptr > decode_sigs( - session().get_decode_signals()); - vector< shared_ptr > traces( - sigs.size() + decode_sigs.size()); -#else - vector< shared_ptr > traces(sigs.size()); -#endif - - auto i = traces.begin(); - i = copy(sigs.begin(), sigs.end(), i); -#ifdef ENABLE_DECODE - i = copy(decode_sigs.begin(), decode_sigs.end(), i); -#endif - - stable_sort(traces.begin(), traces.end(), compare_trace_v_offsets); - return traces; -} - -list > View::selected_items() const -{ - list > items; - - // List the selected signals - const vector< shared_ptr > traces(get_traces()); - for (shared_ptr t : traces) { - assert(t); - if (t->selected()) - items.push_back(t); - } - - // List the selected cursors - if (_cursors.first()->selected()) - items.push_back(_cursors.first()); - if (_cursors.second()->selected()) - items.push_back(_cursors.second()); - - return items; -} - set< shared_ptr > View::get_visible_data() const { - const vector< shared_ptr > sigs( - session().get_signals()); + shared_lock lock(session().signals_mutex()); + const vector< shared_ptr > &sigs(session().signals()); // Make a set of all the visible data objects set< shared_ptr > visible_data; @@ -341,22 +327,6 @@ const QPoint& View::hover_point() const return _hover_point; } -void View::normalize_layout() -{ - const vector< shared_ptr > traces(get_traces()); - - int v_min = INT_MAX; - for (const shared_ptr t : traces) - v_min = min(t->get_v_offset(), v_min); - - const int delta = -min(v_min, 0); - for (shared_ptr t : traces) - t->set_v_offset(t->get_v_offset() + delta); - - verticalScrollBar()->setSliderPosition(_v_offset + delta); - v_scroll_value_changed(verticalScrollBar()->sliderPosition()); -} - void View::update_viewport() { assert(_viewport); @@ -406,9 +376,11 @@ void View::update_scroll() // Set the vertical scrollbar verticalScrollBar()->setPageStep(areaSize.height()); - verticalScrollBar()->setRange(0, - _viewport->get_total_height() + SignalMargin - - areaSize.height()); + + const pair extents = v_extents(); + const int extra_scroll_height = (extents.second - extents.first) / 4; + verticalScrollBar()->setRange(extents.first - extra_scroll_height, + extents.first + extra_scroll_height); } void View::update_layout() @@ -427,12 +399,17 @@ void View::update_layout() update_scroll(); } -bool View::compare_trace_v_offsets(const shared_ptr &a, - const shared_ptr &b) +void View::paint_label(QPainter &p, int right, bool hover) +{ + (void)p; + (void)right; + (void)hover; +} + +QRectF View::label_rect(int right) { - assert(a); - assert(b); - return a->get_v_offset() < b->get_v_offset(); + (void)right; + return QRectF(); } bool View::eventFilter(QObject *object, QEvent *event) @@ -484,6 +461,22 @@ void View::resizeEvent(QResizeEvent*) update_layout(); } +void View::appearance_changed(bool label, bool content) +{ + if (label) + _header->update(); + if (content) + _viewport->update(); +} + +void View::extents_changed(bool horz, bool vert) +{ + _sticky_events |= + (horz ? SelectableItemHExtentsChanged : 0) | + (vert ? SelectableItemVExtentsChanged : 0); + _lazy_event_handler.start(); +} + void View::h_scroll_value_changed(int value) { if (_updating_scroll) @@ -512,16 +505,33 @@ void View::v_scroll_value_changed(int value) void View::signals_changed() { - int offset = SignalMargin + SignalHeight; - const vector< shared_ptr > traces(get_traces()); - for (shared_ptr t : traces) { - t->set_view(this); - t->set_v_offset(offset); - offset += SignalHeight + 2 * SignalMargin; + // Populate the traces + clear_child_items(); + + shared_lock lock(session().signals_mutex()); + const vector< shared_ptr > &sigs(session().signals()); + for (auto s : sigs) + add_child_item(s); + +#ifdef ENABLE_DECODE + const vector< shared_ptr > decode_sigs( + session().get_decode_signals()); + for (auto s : decode_sigs) + add_child_item(s); +#endif + + // Create the initial layout + int offset = 0; + for (shared_ptr r : *this) { + const pair extents = r->v_extents(); + if (r->enabled()) + offset += -extents.first; + r->force_to_v_offset(offset); + if (r->enabled()) + offset += extents.second; } update_layout(); - normalize_layout(); } void View::data_updated() @@ -545,9 +555,23 @@ void View::on_signals_moved() signals_moved(); } -void View::on_geometry_updated() +void View::process_sticky_events() { - update_layout(); + if (_sticky_events & SelectableItemHExtentsChanged) + update_layout(); + if (_sticky_events & SelectableItemVExtentsChanged) { + _viewport->update(); + _header->update(); + } + + // Clear the sticky events + _sticky_events = 0; +} + +void View::on_hover_point_changed() +{ + for (shared_ptr r : *this) + r->hover_point_changed(); } } // namespace view