<div dir="ltr">I will do them all one by one then.<br>I will include time.h in <span style="font-family:arial,sans-serif;font-size:12.8000001907349px">thread.c and any other file that uses the time function.</span><br><br>This does mean that this bug was present in vlc already.<br>It's just HAVE_GMTIME_R and HAVE_LOCALTIME_R were never true for vlc under mingw-w64 before.<br>So the developers never included time.h where they used it<br><br><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">You can do whatever is most convenient for you or your organization within your VLC fork or VLC patch series. But upstream, you get to fix bugs properly, even if it means one by one.</span></blockquote><br>This is not for another fork of vlc this is so that you guys can build for WinRT under latest trunk<br><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 18, 2014 at 6:41 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-18 19:17, Martell Malone a écrit :<span class=""><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
If I include time.h in thread.c the error appears somewhere else then<br>
if I patch that it appears somewhere else again and so on.<br>
This is why I changed vlc_fixups.h to fix them all.<br>
<br>
Im not sure if there is another solution but<br>
I can go through them all 1 by 1 if you want ?<br>
It will be time consuming is all.<br>
<br>
Either way I need to get this fixed. :)<br>
</blockquote>
<br></span>
You can do whatever is most convenient for you or your organization within your VLC fork or VLC patch series. But upstream, you get to fix bugs properly, even if it means one by one.<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>