<div dir="ltr"><div><div><div><div><div><div><blockquote style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex" class="gmail_quote">reeaddrinfo() is tied to getaddrinfo(). Testing for it seems a bit odd: 
the result is unlikely to be valid if you have compat freeaddrinfo() 
with platform-provided getaddrinfo(). Also currently 
<vlc_network.h> assumes freeaddrinfo() is available on eveyr 
platform but OS/2.<br></blockquote><span dir="ltr"></span>I was just told this patch was done before the OS/2 specific change was made.<br></div></div><span dir="ltr">and that point of checking for things is to avoid encoding platform specific prior knowledge</span><br>So it would be a cleaner solution?<br><br></div></div>By going the hard coded route the WINRT app uses the version provided with vlc and not the one windows provides from vista+.<br></div><div>Essentially you both created fixes for the same bug that appeared I would just prefer to use the detection because of the WinRT app.<br></div><div><br></div>On the localtime_r and gmtime_r thing.<br></div>I'm trying my best second time around to get them to change back to functions rather than removing detection<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Jan 14, 2015 at 8:27 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 2015-01-14 16:15, 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="">
Apologies I hit reply not realizing it missed the mailing list<br>
<br>
 <br>
<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Fair enough. The code still makes no sense to me.<br>
</blockquote>
<br></span>
What about it dont you understand ?<br>
</blockquote>
<br>
freeaddrinfo() is tied to getaddrinfo(). Testing for it seems a bit odd: the result is unlikely to be valid if you have compat freeaddrinfo() with platform-provided getaddrinfo(). Also currently <vlc_network.h> assumes freeaddrinfo() is available on eveyr platform but OS/2.<div class="HOEnZb"><div class="h5"><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>
</div></div></blockquote></div><br></div>