<div dir="ltr"><div class="gmail_default" style="font-size:small">Checked patch into GIT after testing before and after as recommended.  It must be Slackware specific because it did not crash before on Fedora 64-bit - or maybe 32-bit but anyway it is an improvement.  I still will be doing more testing tomorrow with additional un-do's to stress it a bit more.<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 26, 2022 at 12:59 PM Andrew Randrianasulu <<a href="mailto:randrianasulu@gmail.com">randrianasulu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">пн, 26 дек. 2022 г., 22:54 Phyllis Smith <<a href="mailto:phylsmith2017@gmail.com" target="_blank">phylsmith2017@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small">I will test that but I am not very good at testing and was hoping from feedback from someone who is better at it.  Will start testing that in about 30 minutes.<br></div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto">I tested ot like this.</div><div dir="auto"><br></div><div dir="auto">load some jpg image, it will expand itself as default 3 sec long timeline.</div><div dir="auto"><br></div><div dir="auto">"Save as" with xml extension.</div><div dir="auto"><br></div><div dir="auto">Load this xml as nested.</div><div dir="auto"><br></div><div dir="auto">hit 'undo' right away.</div><div dir="auto"><br></div><div dir="auto">for me  cingg crashes :(</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small"></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Dec 26, 2022 at 12:17 PM Andrew Randrianasulu <<a href="mailto:randrianasulu@gmail.com" rel="noreferrer" target="_blank">randrianasulu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">вс, 20 нояб. 2022 г., 01:23 Phyllis Smith <<a href="mailto:phylsmith2017@gmail.com" rel="noreferrer" target="_blank">phylsmith2017@gmail.com</a>>:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small"><u>IgorBeghetto:</u><br></div><div class="gmail_default" style="font-size:small">I created an AppImage that contains the BT #558 bug fix (0x00 vs. 0xff) AND with these 2 Experimental patches in to fix "nested edl undo crash" so you could test if you find the time.  The fix for 558 is the most important though and you already built that.  It is at:</div><div class="gmail_default" style="font-size:small">   <a href="https://cinelerra-gg.org/download/testing/test_ubuntu16.AppImage" rel="noreferrer noreferrer" target="_blank">https://cinelerra-gg.org/download/testing/test_ubuntu16.AppImage</a></div><div class="gmail_default" style="font-size:small"><br></div><div class="gmail_default" style="font-size:small">Locking and unlocking are extremely sensitive and can cause all kinds of weird crashes so I guess that is why the patch is called EXPERIMENTAL.  But could definitely be the missing part and fix the problem.</div></div></blockquote></div></div><div dir="auto"><br></div><div dir="auto">I recompiled cingg on my Slackware without those two patches and sadly crash on nested edl load/undo still around :(</div><div dir="auto"><br></div><div dir="auto">Phyllis, can you just add part with checking existence of edl, and may be leave locking changes out for now? ( I thought we should unlock in same function if we exit from it but may be I am wrong ...)</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div class="gmail_default" style="font-size:small"><br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Nov 13, 2022 at 6:50 PM Andrew Randrianasulu <<a href="mailto:randrianasulu@gmail.com" rel="noreferrer noreferrer" target="_blank">randrianasulu@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="auto"></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">---------- Forwarded message ---------<br>От: <strong class="gmail_sendername" dir="auto">Андрей Рандрианасулу</strong> <span dir="auto"><<a href="mailto:randrik@mail.ru" rel="noreferrer noreferrer" target="_blank">randrik@mail.ru</a>></span><br>Date: пн, 14 нояб. 2022 г., 04:37<br>Subject: oh, my fix for nested edl undo crash regressed  video!<br>To: randrianasulu <<a href="mailto:randrianasulu@gmail.com" rel="noreferrer noreferrer" target="_blank">randrianasulu@gmail.com</a>><br></div><br><br>
<div><div>better (?) fix (tested with video this time)</div><div> </div><div><div><div>--<br>Андрей  Рандрианасулу</div></div></div></div>
</div>
</blockquote></div>
</blockquote></div></div></div>
</blockquote></div>
</blockquote></div></div></div>
</blockquote></div>