From 17c7f31c7bb674d5017a8de66a5816e34542abb8 Mon Sep 17 00:00:00 2001 From: Soeren Apel Date: Fri, 26 May 2017 11:34:31 +0200 Subject: [PATCH] AnalogSignal: Only report an extent change when there is one --- pv/view/analogsignal.cpp | 13 +++++++------ 1 file changed, 7 insertions(+), 6 deletions(-) diff --git a/pv/view/analogsignal.cpp b/pv/view/analogsignal.cpp index 63e759f..1067355 100644 --- a/pv/view/analogsignal.cpp +++ b/pv/view/analogsignal.cpp @@ -608,13 +608,17 @@ void AnalogSignal::perform_autoranging(bool keep_divs, bool force_update) // If there is still no positive div when we need it, add one // (this can happen when pos_vdivs==neg_vdivs==0) - if ((max > 0) && (pos_vdivs_ == 0)) + if ((max > 0) && (pos_vdivs_ == 0)) { pos_vdivs_ = 1; + owner_->extents_changed(false, true); + } // If there is still no negative div when we need it, add one // (this can happen when pos_vdivs was 0 or 1 when trying to split) - if ((min < 0) && (neg_vdivs_ == 0)) + if ((min < 0) && (neg_vdivs_ == 0)) { neg_vdivs_ = 1; + owner_->extents_changed(false, true); + } double min_value_per_div; if ((pos_vdivs_ > 0) && (neg_vdivs_ > 0)) @@ -723,11 +727,8 @@ void AnalogSignal::on_samples_added() { perform_autoranging(false, false); - if (owner_) { - // Call order is important, otherwise the lazy event handler won't work - owner_->extents_changed(false, true); + if (owner_) owner_->row_item_appearance_changed(false, true); - } } void AnalogSignal::on_pos_vdivs_changed(int vdivs) -- 2.30.2