---
pv/mainwindow.cpp | 3 ++-
pv/view/view.cpp | 3 ++-
2 files changed, 4 insertions(+), 2 deletions(-)
diff --git a/pv/mainwindow.cpp b/pv/mainwindow.cpp
index f7c4712..d655374 100644
--- a/pv/mainwindow.cpp
+++ b/pv/mainwindow.cpp
@@ -280,8 +280,9 @@ void MainWindow::remove_session(shared_ptr<Session> session)
// that Qt keeps a pointer to the view around
entry.second->setParent(0);
- // Remove this entry from the container
+ // Remove this entry from the container and
stop iterating.
view_docks_.erase(entry.first);
+ break;
}
}
diff --git a/pv/view/view.cpp b/pv/view/view.cpp
index a086d23..9751f00 100644
--- a/pv/view/view.cpp
+++ b/pv/view/view.cpp
@@ -155,7 +155,8 @@ View::View(Session &session, QWidget *parent) :
cursors_(new CursorPair(*this)),
next_flag_text_('A'),
trigger_markers_(),
- hover_point_(-1, -1)
+ hover_point_(-1, -1),
+ scroll_needs_defaults(false)
{
connect(scrollarea_.horizontalScrollBar(), SIGNAL(valueChanged(int)),
this, SLOT(h_scroll_value_changed(int)));
--
2.5.5
On Sat, Dec 24, 2016 at 07:24:48PM +0100, Soeren Apel wrote:
> Hi Pettu,
>
> thanks for the fixes, it's always nice if others help out =)
>
> > diff --git a/pv/mainwindow.cpp b/pv/mainwindow.cpp
> > index f7c4712..9e82fc8 100644
> > --- a/pv/mainwindow.cpp
> > +++ b/pv/mainwindow.cpp
> > @@ -269,8 +269,8 @@ void
> > MainWindow::remove_session(shared_ptr<Session> session)
> >
> > for (shared_ptr<views::ViewBase> view : session->views()) {
> > // Find the dock the view is contained in and remove
> > it
> > - for (auto entry : view_docks_)
> > - if (entry.second == view) {
> > + for (auto it = view_docks_.begin(); it !=
> > view_docks_.end(); )
> > + if (it->second == view) {
> >
>
> I see the issue here now, nice catch! Instead of using an iterator
> though, I would prefer a more straightforward approach: simply
> break-ing from the inner loop when a match was found. There can be
> only a single match at most, so there's no point continuing after
> finding the only match. Would you mind supplying a patch that
> does this instead?
>
>
> > diff --git a/pv/view/view.cpp b/pv/view/view.cpp
> > index a086d23..9751f00 100644
> > --- a/pv/view/view.cpp
> > +++ b/pv/view/view.cpp
> > @@ -155,7 +155,8 @@ View::View(Session &session, QWidget *parent) :
> > cursors_(new CursorPair(*this)),
> > next_flag_text_('A'),
> > trigger_markers_(),
> > - hover_point_(-1, -1)
> > + hover_point_(-1, -1),
> > + scroll_needs_defaults(false)
> > {
> > connect(scrollarea_.horizontalScrollBar(),
> > SIGNAL(valueChanged(int)),
> > this, SLOT(h_scroll_value_changed(int)));
>
>
> That's good to go in.
>
> Feel free to report any other issues/errors you encounter :)
>
> All the best,
> -Soeren
>
------------------------------------------------------------------------------
Developer Access Program for Intel Xeon Phi Processors
Access to Intel Xeon Phi processor-based developer platforms.
With one year of Intel Parallel Studio XE.
Training and support from Colfax.
Order your platform today.http://sdm.link/intel
_______________________________________________
sigrok-devel mailing list
sigrok-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/sigrok-devel
