<div class="gmail_quote">On Fri, Jan 29, 2010 at 12:33 PM, git version control <span dir="ltr"><<a href="mailto:git@videolan.org">git@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
vlc | branch: master | Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org">jb@videolan.org</a>> | Fri Jan 29 12:31:42 2010 +0100| [5ebb3da913cfe7aa14201eb86321cd5a9dc7978d] | committer: Jean-Baptiste Kempf<br>
<br>
Qt: try to avoid a potential race condition<br>
<br>
If we receive a currentChanged, and currentView ain't set, we might crash in<br>
handleExpansion<br>
<br>
Crashes have happened at<br>
StandardPLPanel::handleExpansion (this=<value optimized out>, index=@0x406b56a0) at<br>
components/playlist/standardpanel.cpp<br><br></blockquote><div><br><br>I would like to clarify thing out to myself. I wonder, was that commit a direct response of a crash that you experienced? I know that crashes _happened_ at handleExpansion() but that was a while ago before I fixed the bug that currentView was not set in StandardPLPanel constructor _at all_. I believe that as long as it is set at any point in constructor, crashes shouldn't happen, because I believe that in VLC's case all our events from core are handled in the main Qt thread, so handleExpansion() will never happen before the end of StandardPLPanel constructor.<br>
Is that so? ...just for the sake of calirification.<br><br>Best regards<br></div></div>