<div dir="ltr">Ahh you merged it 2 hours ago?<br>Right before I sent the mail.<br><br>Cheers :)</div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Nov 28, 2014 at 1:42 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Wasn't this already merged?<br>
<br>
On 28 Nov, Martell Malone wrote :<br>
<div class="HOEnZb"><div class="h5">> ping ?<br>
> Is this patch good to merge or do I need to make changes ?<br>
><br>
> Kind Regards<br>
><br>
> On Tue, Nov 25, 2014 at 12:24 PM, Martell Malone <<a href="mailto:martellmalone@gmail.com">martellmalone@gmail.com</a>><br>
> wrote:<br>
><br>
> > In that case.<br>
> ><br>
> > Updated patch<br>
> ><br>
> > From 1c621a9c7dafe365623bc2c7199a52736f0a3983 Mon Sep 17 00:00:00 2001<br>
> > From: Martell Malone <<a href="mailto:martellmalone@gmail.com">martellmalone@gmail.com</a>><br>
> > Date: Tue, 25 Nov 2014 12:22:09 +0000<br>
> > Subject: [PATCH] fix some time includes<br>
> ><br>
> > ---<br>
> >  modules/demux/mkv/mkv.hpp | 4 +---<br>
> >  src/win32/thread.c        | 1 +<br>
> >  2 files changed, 2 insertions(+), 3 deletions(-)<br>
> ><br>
> > diff --git a/modules/demux/mkv/mkv.hpp b/modules/demux/mkv/mkv.hpp<br>
> > index 8c5af80..392b9e7 100644<br>
> > --- a/modules/demux/mkv/mkv.hpp<br>
> > +++ b/modules/demux/mkv/mkv.hpp<br>
> > @@ -45,9 +45,7 @@<br>
> >  #include <vlc_common.h><br>
> >  #include <vlc_plugin.h><br>
> ><br>
> > -#ifdef HAVE_TIME_H<br>
> > -#   include <time.h>                                               /*<br>
> > time() */<br>
> > -#endif<br>
> > +#include <time.h><br>
> ><br>
> >  #include <vlc_meta.h><br>
> >  #include <vlc_charset.h><br>
> > diff --git a/src/win32/thread.c b/src/win32/thread.c<br>
> > index 7dcfc4c..9bc787d 100644<br>
> > --- a/src/win32/thread.c<br>
> > +++ b/src/win32/thread.c<br>
> > @@ -37,6 +37,7 @@<br>
> >  #include <assert.h><br>
> >  #include <limits.h><br>
> >  #include <errno.h><br>
> > +#include <time.h><br>
> ><br>
> >  /*** Static mutex and condition variable ***/<br>
> >  static vlc_mutex_t super_mutex;<br>
> > --<br>
> > 2.1.3<br>
> ><br>
> > On Tue, Nov 25, 2014 at 12:18 PM, Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>><br>
> > wrote:<br>
> ><br>
> >> Le 2014-11-25 15:05, Martell Malone a écrit :<br>
> >><br>
> >>> #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>
> >>>><br>
> >>><br>
> >>> at the start so I dont understand why time.h is being missed here ?<br>
> >>><br>
> >><br>
> >> Because nothing defines HAVE_TIME_H. The ifdef is bogus.<br>
> >><br>
> >><br>
> >> --<br>
> >> Rémi Denis-Courmont<br>
> >> _______________________________________________<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/listinfo/vlc-devel</a><br>
> >><br>
> ><br>
> ><br>
<br>
> _______________________________________________<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/listinfo/vlc-devel</a><br>
<br>
<br>
</div></div><span class="HOEnZb"><font color="#888888">--<br>
With my kindest regards,<br>
</font></span><span class="im HOEnZb"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
</span><div class="HOEnZb"><div class="h5">_______________________________________________<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/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br></div>