Thanks!<br><br><div class="gmail_quote">On Tue, Nov 30, 2010 at 10:33 PM, Hugo Beauzee-Luyssen <span dir="ltr"><<a href="mailto:beauze.h@gmail.com">beauze.h@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">

<div><div></div><div class="h5">On Mon, Oct 11, 2010 at 6:55 PM, Rohit Yadav <<a href="mailto:rohityadav89@gmail.com">rohityadav89@gmail.com</a>> wrote:<br>
> Hello,<br>
><br>
> This commit fixes the setOpacity-crash bug as reported by me earlier... but<br>
> INTRODUCES A BUG!<br>
><br>
> BUG:<br>
> Select a clip and hold/move/release the clip by mouse and at the same time<br>
> press delete, VLMC blows off.<br>
><br>
> INTERNALS:<br>
> When the clip is selected that particular (AbstractGraphicsItem) item<br>
> pointer is assigned to m_actionItem,<br>
> now if someone pressed delete button on keyboard to delete this item, it<br>
> calls removeItem which deletes this item.<br>
> But m_actionItem is still not NULL... so when the user moves/releases the<br>
> mouse... the asserts for m_actionItem<br>
> won't work as m_actionItem is still not NULL and holds the address of the<br>
> deleted heap/memory.<br>
><br>
> REMEDY:<br>
> m_actionItem is the item that is currently selected, so check with the item<br>
> to be deleted and make it NULL, if so.<br>
> BTW, item is always m_actionItem so perhaps no need to check it (still I did<br>
> just in case) except when the layer is changed... discussed below.<br>
><br>
>>   TracksView::removeItem( AbstractGraphicsItem *item )<br>
>>  {<br>
>> +    // Is it the same item captured by mouse events<br>
>> +    if( item == m_actionItem )<br>
>> +        m_actionItem = NULL;<br>
>> +<br>
><br>
> INTRODUCED BUG:<br>
> Now, if the selected clip item is moved to some other than the original<br>
> layer, and same behavior is repeated,<br>
> (clip is deleted while it is being held/moved) it fails to delete the clip<br>
> and at the same time add the action in Undo/Redo.<br>
> PLEASE fix this, layer-move-deletion bug... :)<br>
><br>
> Cheers,<br>
> Rohit Yadav<br>
><br>
<br>
</div></div>Hello again Rohit,<br>
<br>
Sorry this was lost in my mailbox abysses. It has been fixed in<br>
5b0a80573fe0cf8296f28e38f3c7ed1a9e8f18fc<br>
<br>
Cheers,<br>
<font color="#888888"><br>
--<br>
Hugo Beauzée-Luyssen<br>
</font></blockquote></div><br>