<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="12" style="border: 1px #c9c399 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="https://git.reviewboard.kde.org/r/128874/">https://git.reviewboard.kde.org/r/128874/</a>
</td>
</tr>
</table>
<br />
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Did you get a solution to this problem? The reason I ask is that there is still a problem lurking in this area.
Having selected a profile, I then go to select the file to import. Having got to the file selector, if I then decide instead to reverse to get a different profile, I cancel the file selector and am returned to the intro profile screen. If I cancel that too, when KMM is terminated, it aborts - Segmentation fault (core dumped). I've spent quite a bit of time on this, making sure everything that should be deleted, is deleted, but don't yet have a solution. As in this review, the Fileselect dialog is the crux, commenting it out avoids the abort.</p></pre>
<br />
<p>- Allan Anderson</p>
<br />
<p>On September 16th, 2016, 4:38 p.m. UTC, Łukasz Wojniłowicz wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="12" style="border: 1px #888a85 solid; border-radius: 6px; -moz-border-radius: 6px; -webkit-border-radius: 6px;">
<tr>
<td>
<div>Review request for KMymoney.</div>
<div>By Łukasz Wojniłowicz.</div>
<p style="color: grey;"><i>Updated Sept. 16, 2016, 4:38 p.m.</i></p>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt;">Repository: </b>
kmymoney
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;"><p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">After closing CSV Importer in the middle and then KMyMoney, I get "The program has unexpectedly finished".
The problem doesn't occur if CSV Importer goes all way through to the last page; then I can go back and close it wherever I want.
If I comment out this line, there is no problem at all.</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;"><div class="codehilite" style="background: #f8f8f8"><pre style="line-height: 125%"><span></span> QPointer<span style="color: #666666"><</span>QFileDialog<span style="color: #666666">></span> dialog <span style="color: #666666">=</span> <span style="color: #008000; font-weight: bold">new</span> QFileDialog(<span style="color: #008000; font-weight: bold">this</span>, QString(),
fileInfo.absoluteFilePath(),
i18n(<span style="color: #BA2121">"*.csv *.PRN *.txt | CSV Files</span><span style="color: #BB6622; font-weight: bold">\n</span><span style="color: #BA2121"> *|All files"</span>));
</pre></div>
</p>
<p style="padding: 0;text-rendering: inherit;margin: 0;line-height: inherit;white-space: inherit;">Memory on which dialog pointed wasn't deleted in the method and it obviously need to be deleted, but the problem remains. Does anyone know how to prevent QtCreator from showing "The program has unexpectedly finished" here?</p></pre>
</td>
</tr>
</table>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kmymoney/plugins/csvimport/csvwizard.h <span style="color: grey">(ecec5b0)</span></li>
<li>kmymoney/plugins/csvimport/csvwizard.cpp <span style="color: grey">(b576dea)</span></li>
</ul>
<p><a href="https://git.reviewboard.kde.org/r/128874/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>