<meta http-equiv="content-type" content="text/html; charset=utf-8"><span class="Apple-style-span" style="font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">Hi Jan!<div><br></div><div>Thanks for the comments!<br>
<br><div class="gmail_quote"><div class="im" style="color: rgb(80, 0, 80); ">On 18 October 2010 04:32, Jan Kundrát <span dir="ltr"><<a href="mailto:jkt@gentoo.org" target="_blank" style="color: rgb(119, 153, 187); ">jkt@gentoo.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
Hi Andreas, thanks for your patch, unfortunately I can't really commit it, reasons are below.<br><br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
+    // check for the special entry for self-defined<br>+    if (name == i18n("Your Command Line")) {<br>+<br>+        static RunDialog* dialog;<br>+        dialog = new RunDialog(MainWindow::Window::theMainWindow(), _list);<br>
+        dialog->show();<br>+<br>+        return;<br>+    }<br></blockquote><br>Sorry, but this is just ugly. I haven't checked the code (sorry for that), but please do the check based on the QAction* action's value and not the actual string data (that might involve keeping track of the pointer value's somewhere if the list of QActions is somehow runtime-generated). I realize that's from the old code, but I still do not like it at all.<br>
<br>Also note that your code leaks memory, unless there's a proper deleteLater() connected to the dialog's slots (and if it was, current code would be broken). Current version won't leak, as the constructor is called exactly once (the GUI runs in one thread, so that's not an issue here), while you call it each time it's needed, not destroying the dialog upon its completion. If you go that way, there's also no need to define the local variable as static, either.<br>
</blockquote><div><br></div></div><div>I have reworked the code so that no memory leaks any more by setting the image list anew each time the dialog is used. I will not clean up current code, thats for another time. </div>
<div class="im" style="color: rgb(80, 0, 80); "><div> </div><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
<br><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
-       return;  //user clicked the title entry. (i.e: "All Selected Items")<br>+        return;  //user clicked the title entry. (i.e: "All Selected Items")<br></blockquote><br>Please do not mix whitespace cleanup/changes with real changes.<br>
</blockquote><div><br></div></div><div>Okay.</div><div><br></div><div>Andi</div></div></div></span>