<div dir="ltr">Your point here is very valid on the 4 cases.<br>Number 4 is actually the case being used on windows which is why I get an error.<br><br>I think a #define of the function that cast's it to unsigned int * in mingw64 would be a good solution to cater for both cases.<br>I created a post on the mingw-w64 mailing list linking to this to see about them accepting a commit of wrapper on their end. <br><br>Kindest Regards<br>Martell<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 4, 2014 at 1:25 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-04 15:03, Martell Malone a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">
The issue here is there is nothing wrong with mingw-w64 as they link<br>
to msvcrt<br>
As defined here <a href="http://msdn.microsoft.com/en-us/library/a6xkkxfz.aspx" target="_blank">http://msdn.microsoft.<u></u>com/en-us/library/a6xkkxfz.<u></u>aspx</a><br></span>
[3]<span class=""><br>
<br>
On little endian we can just ignore the warning but depending in the<br>
strictness of the build / package system this is not an option.<br>
</span></blockquote>
<br>
I can conceive four cases:<br>
<br>
1) lfind() is inline,<br>
2) lfind() is statically linked and LTO is active,<br>
3) lfind() is statically linked but LTO is inactive,<br>
4) lfidn() is dynamically linked.<br>
<br>
The pointer aliasing violation is only a potential problem in cases 1 and 2. But binary compatibility with the CRT is only a concern in case 4. And even then, a small wrapper that converts back-and-forth would not be hard.<br>
<br>
So I fail to see an excuse. I'm totally not merging that ugly ifdef hack.<span class="HOEnZb"><font color="#888888"><br>
<br>
-- <br>
Rémi Denis-Courmont</font></span><div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div>