<div dir="ltr">Thanks. I've also removed the Q_ASSERTs (as they aren't right per your code) and pushed.<div><br></div><div>Jose</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Feb 12, 2015 at 9:25 AM, Tapani Pälli <span dir="ltr"><<a href="mailto:tapani.palli@intel.com" target="_blank">tapani.palli@intel.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Patch fixes a crash if impatient user (like me) makes a search<br>
before trace has been loaded.<br>
<br>
Signed-off-by: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>><br>
---<br>
 gui/mainwindow.cpp | 8 ++++++++<br>
 1 file changed, 8 insertions(+)<br>
<br>
diff --git a/gui/mainwindow.cpp b/gui/mainwindow.cpp<br>
index a503b5d..ff89ad7 100644<br>
--- a/gui/mainwindow.cpp<br>
+++ b/gui/mainwindow.cpp<br>
@@ -1118,6 +1118,10 @@ void MainWindow::slotSearchNext(const QString &str,<br>
<br>
     Q_ASSERT(call || frame);<br>
     if (!frame) {<br>
+        // Trace is still loading.<br>
+        if (!call) {<br>
+            return;<br>
+        }<br>
         frame = call->parentFrame();<br>
     }<br>
     Q_ASSERT(frame);<br>
@@ -1133,6 +1137,10 @@ void MainWindow::slotSearchPrev(const QString &str,<br>
<br>
     Q_ASSERT(call || frame);<br>
     if (!frame) {<br>
+        // Trace is still loading.<br>
+        if (!call) {<br>
+            return;<br>
+        }<br>
         frame = call->parentFrame();<br>
     }<br>
     Q_ASSERT(frame);<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.1.0<br>
<br>
_______________________________________________<br>
apitrace mailing list<br>
<a href="mailto:apitrace@lists.freedesktop.org">apitrace@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/apitrace" target="_blank">http://lists.freedesktop.org/mailman/listinfo/apitrace</a><br>
</font></span></blockquote></div><br></div>