<div dir="ltr"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><span style="font-family:arial,sans-serif;font-size:12.8000001907349px">This patch manages to violate the spec, break the build and fix nothing (on little endian), all at once.</span></blockquote><br>As defined in <span style="font-family:arial,sans-serif;font-size:12.8000001907349px">mingw-w64-headers/crt/search.h<br></span><div>_CRTIMP void *__cdecl _lfind(const void *_Key,const void *_Base,unsigned int *_NumOfElements,unsigned int _SizeOfElements,int (__cdecl *_PtFuncCompare)(const void *,const void *</div><div>));<br><br>unsigned int *_NumOfElements; <br><br>I already discussed this with the mingw-w64 devs and they said the must comply with what msvcrt does because you can only have one function definition with one name in C.<br>Their advice is for vlc to make an exception when using lfind.<br><br>Maybe I am missing something but could you explain how this patch breaks the build ?<br>And / Or recommend some changes to it in terms of spec.<br> <br>This is my first patch so forgive me if it doesn't conform</div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Nov 3, 2014 at 12:42 PM, Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net" target="_blank">remi@remlab.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Le 2014-11-03 15:30, Martell Malone a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
MS dont conform to the size_t argument and always use unsigned int<span class=""><br>
even on x64 <br>
This was applied to 2.2 branch but should also be able to go onto HEAD<br>
</span></blockquote>
<br>
This patch manages to violate the spec, break the build and fix nothing (on little endian), all at once.<br>
<br>
Nack.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Rémi Denis-Courmont<br>
______________________________<u></u>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>listinfo/vlc-devel</a><br>
</font></span></blockquote></div><br></div>