<div dir="ltr">Thanks for your remarks. <div><br></div><div>You are right, there is no reason to change the parent from "this" to </div><div>NULL. I made the changes and attached a new version of the patch.</div><div><br></div><div>Claudiu</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Mar 21, 2015 at 3:55 PM, Lubomir I. Ivanov <span dir="ltr"><<a href="mailto:neolit123@gmail.com" target="_blank">neolit123@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 20 March 2015 at 23:06, Claudiu Olteanu<br>
<span class=""><<a href="mailto:olteanu.vasilica.claudiu@gmail.com">olteanu.vasilica.claudiu@gmail.com</a>> wrote:<br>
> Hi there,<br>
><br>
> Here is a patch which fixes the #844 ticket.<br>
<br>
</span>lines 31 and 38 of the patch contain trailing whitespace.<br>
before committing you can use:<br>
git diff --check<br>
<br>
- filename = QFileDialog::getSaveFileName(this, tr("Save file as"),<br>
default_filename,<br>
- tr("Subsurface XML files (*.ssrf *.xml *.XML)"));<br>
<br>
(1) i know the original wasn't either...<br>
<br>
+<br>
+ QFileDialog selection_dialog(NULL, tr("Save file as"), default_filename,<br>
+ tr("Subsurface XML files (*.ssrf *.xml *.XML)"));<br>
+<br>
<br>
(2)...but the second line - tr(....) should be preferable indented<br>
with spaces like so:<br>
(should be viewed with a monospace font)<br>
<br>
QFileDialog selection_dialog(NULL, tr("Save file as"), default_filename,<br>
                             tr("Subsurface XML files (*.ssrf *.xml *.XML)"));<br>
<br>
or with a single tab.<br>
<br>
NULL for parent in QFileDialog seems to work, but any reason to change that?<br>
everywhere else in mainwindow.cpp we use "this".<br>
<br>
other than these two small remarks, the change seems about right.<br>
<br>
lubomir<br>
--<br>
</blockquote></div><br></div>