<div class="gmail_extra">Hi Robin,</div><div class="gmail_extra"><br></div><div class="gmail_extra">thanks for the feedback.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I'm not sure how to implement a really meaningful unit test for this Mac OS specific problem.</div>
<div class="gmail_extra">Maybe you could have a look at patch set 1 which tries to solve this on the cocoa level.</div><div class="gmail_extra"><br></div><div class="gmail_extra">But I guess this is the wrong list for further discussion anyway.</div>
<div class="gmail_extra"><br></div><div class="gmail_extra">Nils</div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Apr 25, 2012 at 10:28 AM, Robin Burchell <span dir="ltr"><<a href="mailto:robin+qt@viroteck.net" target="_blank">robin+qt@viroteck.net</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
2012/4/25 Nils Jeisecke <<a href="mailto:njeisecke@saltation.de">njeisecke@saltation.de</a>>:<br>
<div class="im">> I case it's not too late, would some consider<br>
> reviewing <a href="https://codereview.qt-project.org/#change,24075" target="_blank">https://codereview.qt-project.org/#change,24075</a> for 4.8.2?<br>
<br>
</div>As noted on the review, you're missing unit tests, which would<br>
personally make me wary of slipping this in without some pretty<br>
intensive testing at this stage. It may be too late for that.<br>
<br>
Your bug report notes that this also affects 4.7.4 and all 4.8<br>
releases so far, so I'd be inclined to let it go into 4.8.3 - which,<br>
given the past two releases, won't be far away.<br>
<br>
BR,<br>
<br>
Robin<br>
</blockquote></div><br><br clear="all"><div><br></div>
</div>