[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