[vlc-devel] [PATCH 1/3] WinRT: define a simple WINRT define than WINAPI_FAMILY_APP

Rémi Denis-Courmont remi at remlab.net
Tue Jan 8 14:31:45 CET 2013


On Tue,  8 Jan 2013 13:44:34 +0100, Jean-Baptiste Kempf <jb at videolan.org>
wrote:
> This will allow us to use #ifdef(WINRT) in the code

Comment is wrong.

> ---
>  include/vlc_common.h | 8 ++++++++
>  1 file changed, 8 insertions(+)
> 
> diff --git a/include/vlc_common.h b/include/vlc_common.h
> index d09bf78..399c884 100644
> --- a/include/vlc_common.h
> +++ b/include/vlc_common.h
> @@ -408,6 +408,14 @@ typedef int ( * vlc_callback_t ) ( vlc_object_t *, 

>  /* variable's object */
>   * OS-specific headers and thread types
>  
*****************************************************************************/
>  #if defined( WIN32 )
> +#   if defined(WINAPI_FAMILY) && WINAPI_FAMILY == WINAPI_FAMILY_APP
> +    /* See
> +     *
http://msdn.microsoft.com/en-us/library/windows/apps/hh700130.aspx
> +     *
http://msdn.microsoft.com/en-us/library/windows/apps/hh699881.aspx

That page claims a header must be included first.

> +     *
> http://msdn.microsoft.com/en-us/library/windows/apps/xaml/br205757.aspx
> +     **/
> +#      define VLC_WINRT

What's the point anyway? We don't define VLC_WINDOWS and VLC_MACOS...

> +#   endif
>  #   include <malloc.h>
>  #   ifndef PATH_MAX
>  #       define PATH_MAX MAX_PATH

-- 
Rémi Denis-Courmont
Sent from my collocated server



More information about the vlc-devel mailing list