<div dir="ltr">ping ?<br>Is this patch good to merge or do I need to make changes ?<br><br>Kind Regards<br></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 25, 2014 at 12:24 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">In that case.<br><br>Updated patch<br><br><div>From 1c621a9c7dafe365623bc2c7199a52736f0a3983 Mon Sep 17 00:00:00 2001</div><div>From: Martell Malone <<a href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>></div><div>Date: Tue, 25 Nov 2014 12:22:09 +0000</div><div>Subject: [PATCH] fix some time includes</div><div><br></div><div>---</div><div> modules/demux/mkv/mkv.hpp | 4 +---</div><div> src/win32/thread.c        | 1 +</div><div> 2 files changed, 2 insertions(+), 3 deletions(-)</div><div><br></div><div>diff --git a/modules/demux/mkv/mkv.hpp b/modules/demux/mkv/mkv.hpp</div><div>index 8c5af80..392b9e7 100644</div><div>--- a/modules/demux/mkv/mkv.hpp</div><div>+++ b/modules/demux/mkv/mkv.hpp</div><div>@@ -45,9 +45,7 @@</div><div> #include <vlc_common.h></div><div> #include <vlc_plugin.h></div><div> </div><div>-#ifdef HAVE_TIME_H</div><div>-#   include <time.h>                                               /* time() */</div><div>-#endif</div><div>+#include <time.h></div><div> </div><div> #include <vlc_meta.h></div><div> #include <vlc_charset.h></div><span class=""><div>diff --git a/src/win32/thread.c b/src/win32/thread.c</div><div>index 7dcfc4c..9bc787d 100644</div><div>--- a/src/win32/thread.c</div><div>+++ b/src/win32/thread.c</div><div>@@ -37,6 +37,7 @@</div><div> #include <assert.h></div><div> #include <limits.h></div><div> #include <errno.h></div><div>+#include <time.h></div><div> </div><div> /*** Static mutex and condition variable ***/</div><div> static vlc_mutex_t super_mutex;</div><div>-- </div><div>2.1.3</div></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Nov 25, 2014 at 12:18 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-25 15:05, Martell Malone a écrit :<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
#ifdef HAVE_CONFIG_H<br>
# include "config.h"<br>
#endif<br>
#include <inttypes.h><br>
#include <vlc_common.h><br>
#include <vlc_plugin.h><br>
#ifdef HAVE_TIME_H<br>
#   include <time.h>                              <br>
                /* time() */<br>
#endif <br>
</blockquote>
<br></span>
at the start so I dont understand why time.h is being missed here ?<br>
</blockquote>
<br>
Because nothing defines HAVE_TIME_H. The ifdef is bogus.<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>