<html dir="ltr"><head><title></title><style type="text/css">p.MsoNormal,p.MsoNoSpacing{margin:0}</style></head><body style="text-align:left; direction:ltr;"><div>Ok, I've rebased and attached</div><div><br></div><div>Regarding your issue applying the previous copy, I wonder whether you perhaps missed the note in the first post which pointed out that this built on other patches submitted days earlier? I had no issues rebasing onto git master (except an expected clash with the subsequent MMX purge with someone else's deinterlace work).</div><div><br></div><div>The attached rebased patch collection bundles (in order):</div><div> - The initial set of general i420_rgb/i420_yuy2/i422_yuy2 fixes (patches 1-11)</div><div> - The i420_rgb intrinsics buffer overflow fix (patch 12)</div><div> - The i420_rgb/i420_yuy2/i422_yuy2 AVX2 enhancement (patches 13-14)</div><div> - The packetizer/startcode_helper AVX2 enhancement (patch 15)</div><div> - The purging of MMX/MMXEXT/3Dnow (patches 16-25)</div><div> - The conversion of asm to nasm syntax (patches 26-31)</div><div><br></div><div>The first of these was approved of by one person who just had a question about the second patch (removal of what I believe are unused RV24 artifacts, which he may have misunderstood as removing RV24 support), I had responded to clarify but then nothing happened. With the second (buffer overflow patch) there was some discussion, but no merge of the patch...</div><div><br></div><div>I have taken the opportunity of bundling all of this together, to:</div><div> - fixup one patch (fixed a mistake in a larger previous one)</div><div> - fix a few commit message typos and such</div><div><br></div><div>I have also reworked the MMX purge with the deinterlace plugin on top of the now merged work of Janne Grunau</div><div><br></div><div>Oh, I should point out that with the last set, the nasm conversion, which came about from someone suggesting using this for the AVX2 asm (as I then reworked it to use), all I have done is to switch the syntax itself, I have not made any change to the build files (does a hint not need to be given to the compiler to get it to invoke the right assembler?)</div><div><br></div><div>Regards,</div><div>Lyndon</div><div><br></div><div>On Sat, 2019-02-23 at 12:00 -0500, Jean-Baptiste Kempf wrote:</div><blockquote type="cite" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>Hello Lyndon,<br></div><div><br></div><div>Can you repost all the patches, in the correct order? I cannot apply any of them, in my tree.<br></div><div><br></div><div>Best,<br></div><div><br></div><div>On Thu, 31 Jan 2019, at 09:39, jnqnfe@gmail.com wrote:<br></div><blockquote type="cite" id="fastmail-quoted" style="margin:0 0 0 .8ex; border-left:2px #729fcf solid;padding-left:1ex"><div>sigh, so here's v4<br></div><div><br></div><div>v3 did not include a conversion of the RGB15 AVX2 to nasm, while the v2<br></div><div>-> v3 diff patch (sent in case you'd already started reviewing v2) did<br></div><div><br></div><div>I'm perplexed as to how on earth that happened, since the diff patch<br></div><div>was a simple fixup into v2 to produce v3...<br></div><div><br></div><div>_______________________________________________<br></div><div>vlc-devel mailing list<br></div><div>To unsubscribe or modify your subscription options:<br></div><div>https://mailman.videolan.org/listinfo/vlc-devel<br></div><div><br></div><div><b>Attachments:</b><br></div><ul><li>chroma_avx2_v4.patch<br></li></ul></blockquote><div><br></div><div id="sig60240713"><div class="signature">--<br></div><div class="signature">Jean-Baptiste Kempf -  President<br></div><div class="signature">+33 672 704 734<br></div><div class="signature"> <br></div><div class="signature"><br></div></div><div><br></div></blockquote></body></html>