<div dir="ltr">On Thu, Jul 4, 2013 at 5:42 PM, Peter Lohrmann <span dir="ltr"><<a href="mailto:peterl@valvesoftware.com" target="_blank">peterl@valvesoftware.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">





<div lang="EN-US" link="blue" vlink="purple">
<div>
<p class="MsoNormal">I fixed small memory leak / assert if a trace file can't be loaded due to being recorded with a newer version number than qapitrace.</p></div></div></blockquote><div><br></div><div style>Pushed.</div>
<div style> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"> In this scenario, there is no error message that gets shown in the UI, but the title bar is updated with
 the trace file name, so the situation is very confusing. Is there an easy way to address these issues? I haven't used Qt enough yet to fully understand the moc files, etc, so I don't know the easiest way to add an 'error' Qevent that can be emitted back to
 the main window from the parser. Any help (or another patch) would be appreciated!</p></div></div></blockquote><div><br></div><div style>I'm not sure what's the best way to do that. But I suppose that an ordinary message box would be better than no message at all.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><u></u><u></u></p>
<p class="MsoNormal">The version issue came up as I just started looking at the more-thumbnails branch, but it only supports version 4 of the trace files, and master is up to version 5 now. It seems the various branches need to be updated again.<br>
</p></div></div></blockquote><div><br></div><div style>The several branches are not automatically kept up to date with master . But with git, that can be easily achieved doing</div><div style><br></div><div style>  git merge master</div>
<div style><br></div><div style>and fix any conflicts. I did that for for more-thumbnails branch now, so it should work with with recent traces.</div><div style><br></div><div style>Jose</div><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div lang="EN-US" link="blue" vlink="purple"><div><p class="MsoNormal"><u></u></p><p class="MsoNormal"><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">- Peter<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">---<u></u><u></u></p>
<p class="MsoNormal">common/trace_parser.cpp | 2 ++<u></u><u></u></p>
<p class="MsoNormal">1 file changed, 2 insertions(+)<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">diff --git a/common/trace_parser.cpp b/common/trace_parser.cpp<u></u><u></u></p>
<p class="MsoNormal">index 095af67..6013b39 100644<u></u><u></u></p>
<p class="MsoNormal">--- a/common/trace_parser.cpp<u></u><u></u></p>
<p class="MsoNormal">+++ b/common/trace_parser.cpp<u></u><u></u></p>
<p class="MsoNormal">@@ -65,6 +65,8 @@ bool Parser::open(const char *filename) {<u></u><u></u></p>
<p class="MsoNormal">     version = read_uint();<u></u><u></u></p>
<p class="MsoNormal">     if (version > TRACE_VERSION) {<u></u><u></u></p>
<p class="MsoNormal">         std::cerr << "error: unsupported trace format version " << version << "\n";<u></u><u></u></p>
<p class="MsoNormal">+        delete file;<u></u><u></u></p>
<p class="MsoNormal">+        file = NULL;<u></u><u></u></p>
<p class="MsoNormal">         return false;<u></u><u></u></p>
<p class="MsoNormal">     }<u></u><u></u></p>
<p class="MsoNormal">     api = API_UNKNOWN;<u></u><u></u></p>
<p class="MsoNormal">-- <u></u><u></u></p>
<p class="MsoNormal">1.8.1.msysgit.1<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
</div>

<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>
<br></blockquote></div><br></div></div>