[mythtv] Ticket #4270: MythTV on Windows (with MinGW)

Andrei Tanas andrei at tanas.ca
Fri Dec 14 17:20:18 UTC 2007


> Comment(by nigel):
> 
>  1) suseconds_t should now be something like:
>  {{{
>  Index: compat.h
>  ===================================================================
>  --- compat.h    (revision 15161)
>  +++ compat.h    (working copy)
>  @@ -240,6 +240,10 @@
>   // suseconds_t
>   #include <sys/types.h>
> 
>  +#ifdef USING_MINGW
>  +    typedef long long suseconds_t;
>  +#endif
>  +
>   #include "mythconfig.h"
>   #if defined(CONFIG_DARWIN) && ! defined (_SUSECONDS_T)
>           typedef int32_t suseconds_t;   // 10.3 or earlier don't have
> this
>  }}}
>  but could it be int32_t (like Darwin) instead of long long?

I thought it was supposed to be 64-bit. Yes, this is fine, except int32_t is not always defined. Let's use "typedef int suseconds_t". I'll post a patch under #4270.


>  2) For MythMusic, instead of the #ifdefs, it might be better to make a
>  mostly empty cddecoder implementation?
>  ''e.g.''cddecoder-windows.cpp? I can knock one up if you like?

Fine with me, but unless some stuff from cddecoder.h is moved to a private class, we will need a couple of #ifdefs in it. Let me know if this is ok, I'll create a patch.



More information about the mythtv-dev mailing list