From: Diane Trout <diane@caltech.edu>
Date: Wed, 22 Mar 2006 04:44:08 +0000 (+0000)
Subject: spurious mouseReleaseEvent caused a crash
X-Git-Url: http://woldlab.caltech.edu/gitweb/?a=commitdiff_plain;h=76283264bf554fe07858f6a58764d44444a5fcf8;p=mussa.git

spurious mouseReleaseEvent caused a crash
For some reason a mouseReleaseEvent was making it to mussa when
using one of the load dialog boxes. This caused a crash.
This patch causes it to not crash.
Though this patch also seems to sometimes select things for no apparent
reason.
---

diff --git a/qui/seqbrowser/SequenceBrowser.cpp b/qui/seqbrowser/SequenceBrowser.cpp
index 1fac6da..00885dd 100644
--- a/qui/seqbrowser/SequenceBrowser.cpp
+++ b/qui/seqbrowser/SequenceBrowser.cpp
@@ -126,9 +126,11 @@ void SequenceBrowser::mouseMoveEvent( QMouseEvent *e)
 void SequenceBrowser::mouseReleaseEvent( QMouseEvent *e)
 {
   drawingBand = false;
-  rubberBand->hide();
-  QRect r = QRect(bandOrigin, e->pos()).normalized();
-  bandOrigin = r.topLeft();
-
-  selectRegion(r.top(), r.left(), r.bottom(), r.right());
+  if (rubberBand != 0) {
+    rubberBand->hide();
+    QRect r = QRect(bandOrigin, e->pos()).normalized();
+    bandOrigin = r.topLeft();
+  
+    selectRegion(r.top(), r.left(), r.bottom(), r.right());
+  }
 }