<br><br>On Friday, November 19, 2021, Phyllis Smith via Cin <<a href="mailto:cin@lists.cinelerra-gg.org">cin@lists.cinelerra-gg.org</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><span class="gmail_default" style="font-size:small">Checked into GIT this morning.<br></span></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span class="gmail_default" style="font-size:small"></span>00<span class="gmail_default" style="font-size:small">0</span>9-attempt-to-fix-_probe-<wbr>function-memory-leak.patch<span class="gmail_default" style="font-size:small"> = yes</span></div><div>0010-additional-XOpenIM-call-<wbr>in-bcwindowbase.C.patch = yes</div><div>0042-fix-typos-in-libdv-<wbr>ifdefs-in-livevideo.patch = yes<span class="gmail_default" style="font-size:small"></span></div></blockquote></div></div></blockquote><div><br></div><div>ow, 42 probably was already fixed by MatN.. sorry. see our discussion in just closed bug about broken compilation on x86_64</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><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"></blockquote><div> </div><div><span class="gmail_default" style="font-size:small"></span><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><span class="gmail_default" style="font-size:small"></span><span class="gmail_default" style="font-size:small">n</span>ow lets try to define minimal patchset... </div></blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><span class="gmail_default" style="font-size:small"></span> so, in summary... </blockquote></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div><br></div><div>try 2-10, 16, 21, 40-42</div><div><br></div><div>may be 38, 54</div></blockquote><div><span class="gmail_default" style="font-size:small">Minimal patchset has all been checked into GIT after very minimal testing on my part. EXCEPT for 54 because there are very many mjpegtools patches that precede it and I can not tell if there will be user impact for non-Termux builds and not sure how to adequately test.</span> <br></div><div><br></div><div><div style="font-size:small" class="gmail_default"><b>Concerns for Andrew </b>on these 3 sets of patches:</div><div style="font-size:small" class="gmail_default">? 0009 ? is freeing this frame always safe or what if it does not exist? I am thinking it would have already crashed if it is empty.</div></div></div></div></blockquote><div><br></div><div>well, i think my intention was to fix memorybleak you discovered. Until you re-enable more interlace probing (as I accedently did in 0001!)  this codepatch will be unreachable. I run with both patches applied, but I do not load great many files.. </div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><div style="font-size:small" class="gmail_default">? 0010 ? does XSetLocaleModifiers affect future locale settings that might cause issues ? or is it just temporary?</div></div></div></div></blockquote><div><br></div><div>without this check it was exiting right at this place on termux... you might wrap it in ifdefs, but I copied solution from another project, so hopefully it should be safe for normal installs (i can dig fir exact bugreport) </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><div style="font-size:small" class="gmail_default">? 0042 ? I changed all of the "if HAVE_DVf" to "ifdef HAVE_LIBDV" because that is what is used everywhere else ?</div><div style="font-size:small" class="gmail_default">? 0054 ?  +  12, 14, 20, 35 ? is there a really good set of tests for patches in mjpegtools which I have not added in yet ?<br></div></div></div></div></blockquote><div><br></div><div>well, without those patches build was failing for me... ( same for two  more ffmpeg patches.. </div><div><br></div><div>For testing I thought jpeg filetype and background render (with default jpeg) will exercise some aspects of mjpegtools, but I can be wrong.. You can also try to enable denoising for dvd - I think it uses mjpegtools. </div><div><br></div><div>also libbthread itself still not in, not sure if you want it in before or after this month build.. </div><div><br></div><div>how far away you are from re-creating build enviroment on your tablet? </div><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div><br></div><div> </div></div></div>
</blockquote>