<div dir="ltr">Here is the current error<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">C:/msys64/home/Martell/MINGW-packages/mingw-w64-vlc-git/src/vlc/src/win32/thread.c:283:17: error: implicit declaration of function 'time' [-Werror=implicit-function-declaration]<br>                 total = CLOCK_FREQ * time (NULL);</blockquote><div><br>If I include time.h in thread.c the error appears somewhere else then 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>I'm 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>   </div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 18, 2014 at 3:59 PM, Martell Malone <span dir="ltr"><<a href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><span class=""><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"><vlc_fixups.h> does not use the time() function anywhere anyhow.</span></blockquote></span><div><br>I will rebuild without the patch now.<br>I need get error for us to find the solution.<br>You may indeed be right and the fix lies somewhere else.<br>I will be back shortly will the error :)  </div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 18, 2014 at 3:53 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 18:40, Martell Malone a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span>
That does not seem right. By that line of thinking, we would need to<br></span>
include <time.h> everywhere, and yet clearly we dont.<br>
</blockquote><span>
<br>
Yes I understand what you mean it does look strange at first look.<br>
<br>
To me it seems vlc_fixups.h is already making an assumption.<br>
It assumes that if HAVE_GMTIME_R and HAVE_LOCALTIME_R are both<br>
defined then you already have a prototype for the time function.<br>
</span></blockquote>
<br>
No. It assumes that if both gmtime_r() and localtime_r() are present, then <vlc_fixups.h> does not depend on <time.h>.<br>
<br>
<vlc_fixups.h> does not use the time() function anywhere anyhow.<div><div><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>
</div></div></blockquote></div><br></div>