<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=us-ascii">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
{font-family:"Cambria Math";
panose-1:2 4 5 3 5 4 6 3 2 4;}
@font-face
{font-family:Calibri;
panose-1:2 15 5 2 2 2 4 3 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
{margin:0in;
margin-bottom:.0001pt;
font-size:11.0pt;
font-family:"Calibri","sans-serif";}
a:link, span.MsoHyperlink
{mso-style-priority:99;
color:blue;
text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
{mso-style-priority:99;
color:purple;
text-decoration:underline;}
span.EmailStyle17
{mso-style-type:personal-compose;
font-family:"Calibri","sans-serif";
color:windowtext;}
.MsoChpDefault
{mso-style-type:export-only;
font-family:"Calibri","sans-serif";}
@page WordSection1
{size:8.5in 11.0in;
margin:1.0in 1.0in 1.0in 1.0in;}
div.WordSection1
{page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-US" link="blue" vlink="purple">
<div class="WordSection1">
<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. 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!<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></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.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">- Peter<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">---<o:p></o:p></p>
<p class="MsoNormal">common/trace_parser.cpp | 2 ++<o:p></o:p></p>
<p class="MsoNormal">1 file changed, 2 insertions(+)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">diff --git a/common/trace_parser.cpp b/common/trace_parser.cpp<o:p></o:p></p>
<p class="MsoNormal">index 095af67..6013b39 100644<o:p></o:p></p>
<p class="MsoNormal">--- a/common/trace_parser.cpp<o:p></o:p></p>
<p class="MsoNormal">+++ b/common/trace_parser.cpp<o:p></o:p></p>
<p class="MsoNormal">@@ -65,6 +65,8 @@ bool Parser::open(const char *filename) {<o:p></o:p></p>
<p class="MsoNormal"> version = read_uint();<o:p></o:p></p>
<p class="MsoNormal"> if (version > TRACE_VERSION) {<o:p></o:p></p>
<p class="MsoNormal"> std::cerr << "error: unsupported trace format version " << version << "\n";<o:p></o:p></p>
<p class="MsoNormal">+ delete file;<o:p></o:p></p>
<p class="MsoNormal">+ file = NULL;<o:p></o:p></p>
<p class="MsoNormal"> return false;<o:p></o:p></p>
<p class="MsoNormal"> }<o:p></o:p></p>
<p class="MsoNormal"> api = API_UNKNOWN;<o:p></o:p></p>
<p class="MsoNormal">-- <o:p></o:p></p>
<p class="MsoNormal">1.8.1.msysgit.1<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
</body>
</html>