<div class="gmail_quote">On Wed, Jan 27, 2010 at 2:41 PM, Ilkka Ollakka <span dir="ltr"><<a href="mailto:ileoo@videolan.org">ileoo@videolan.org</a>></span> wrote: <blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div class="im">On Wed, Jan 27, 2010 at 02:27:42PM +0100, git version control wrote:<br>
> vlc | branch: master | Jakob Leben <<a href="mailto:jleben@videolan.org">jleben@videolan.org</a>> | Wed Jan 27 12:49:48 2010 +0100| [4fdd32fab86f3d0a430ed6bc8394007c41d38b7b] | committer: Jakob Leben<br>
<br>
> Qt: connect currentChanged signal from model earlier<br>
<br>
<br>
</div>Are you sure that connectin it that early is good idea? as currentView<br>
is NULL in that point and then handleExpansion triggers assert, if<br>
currentChanged happens before currentView is defined.<br></blockquote><div><br><br>Well, the way we have it, all our slots are ever called in the same main Qt application thread. That makes it impossible for the handleExpansion to be called before the end of StandardPLPanel constructor.<br>
<br>Best regards<br></div></div>