<br><br><div class="gmail_quote">2008/12/9 Luka Renko <span dir="ltr"><<a href="mailto:lure@kubuntu.org">lure@kubuntu.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="Ih2E3d">On Tuesday 09 December 2008 10:57:06 Andi Clemens wrote:<br>
> In Qt4, you normally don't need a parent at all, it will be mapped anyway<br>
> when adding to a layout. I usually don't provide parents to Layout and<br>
> Widgets, because it is easier for me to read it that way.<br>
><br>
</div><div class="Ih2E3d">> But since we use switch later on and in case of "NoButtons" don't add the<br>
> buttons to a layout, I need to set a parent in ctor of the buttons,<br>
> otherwise we have a memory leak.<br>
<br>
</div>Good catch!<br>
<div class="Ih2E3d"><br>
> About the plainWidget: Right now the only plugin that uses it (and maybe<br>
> the only one that ever will) is RemoveRedEyes. Since I use buttons on the<br>
> right, I think it is ok to skip the plainWidget as you mentioned in<br>
> 'NoButtons' mode.<br>
<br>
</div>OK, I though so.<br>
<br>
Thanks for reviewing, testing and committing it!</blockquote><div><br>I'm waiting your new plugin code for including it officially to next beta release. I will test if it compile fine under Windows.<br><br>Please commit your code on svn when it ready...<br>
<br>Question : it require a specific external depency ?<br><br>Best<br><br>Gilels</div></div><br>