[MPlayer-dev-eng] [PATCH 2/3] Fix some warnings with Objective-C code and declarations

Xidorn Quan quanxunzhen at gmail.com
Mon Jul 15 02:09:33 CEST 2013


On Sun, Jul 14, 2013 at 8:38 PM, Jiang Jiang <gzjjgod at gmail.com> wrote:

> ---
>  libvo/osx_objc_common.h | 2 +-
>  libvo/osx_objc_common.m | 4 ++--
>  libvo/vo_corevideo.h    | 2 +-
>  libvo/vo_corevideo.m    | 7 ++++---
>  4 files changed, 8 insertions(+), 7 deletions(-)
>
> diff --git a/libvo/osx_objc_common.h b/libvo/osx_objc_common.h
> index a0e49f6..c9cd65f 100644
> --- a/libvo/osx_objc_common.h
> +++ b/libvo/osx_objc_common.h
> @@ -62,7 +62,7 @@
>
>  //window & rendering
>  - (void) preinit;
> -- (void) config:(uint32_t)width:(uint32_t)height:(uint32_t)flags;
> +- (void) configWidth: (uint32_t) width height: (uint32_t) height flags:
> (uint32_t)flags;
>  - (void) drawRect: (NSRect *) bounds;
>  - (void) reshape;
>
> diff --git a/libvo/osx_objc_common.m b/libvo/osx_objc_common.m
> index 17a6fec..35d1e33 100644
> --- a/libvo/osx_objc_common.m
> +++ b/libvo/osx_objc_common.m
> @@ -73,7 +73,7 @@ void vo_osx_uninit(void)
>
>  int vo_osx_config(uint32_t width, uint32_t height, uint32_t flags)
>  {
> -    [oglv config:width:height:flags];
> +    [oglv configWidth:width height:height flags:flags];
>      return 1;
>  }
>
> @@ -183,7 +183,7 @@ void vo_osx_swap_buffers(void)
>         [super dealloc];
>  }
>
> -- (void) config:(uint32_t)width:(uint32_t)height:(uint32_t)flags
> +- (void) configWidth:(uint32_t)width height:(uint32_t)height
> flags:(uint32_t)flags
>  {
>         if (flags & VOFLAG_HIDDEN)
>                 return;
> diff --git a/libvo/vo_corevideo.h b/libvo/vo_corevideo.h
> index e1a7229..8bab3c3 100644
> --- a/libvo/vo_corevideo.h
> +++ b/libvo/vo_corevideo.h
> @@ -56,7 +56,7 @@
>
>  //window & rendering
>  - (void) preinit;
> -- (void) config:(uint32_t)width:(uint32_t)height:(uint32_t)flags;
> +- (void) configWidth: (uint32_t) width height: (uint32_t) height flags:
> (uint32_t)flags;
>  - (void) prepareOpenGL;
>  - (void) render;
>  - (void) reshape;
> diff --git a/libvo/vo_corevideo.m b/libvo/vo_corevideo.m
> index 603c004..b375af7 100644
> --- a/libvo/vo_corevideo.m
> +++ b/libvo/vo_corevideo.m
> @@ -25,6 +25,7 @@
>  #include <sys/mman.h>
>  #include <unistd.h>
>  #include <CoreServices/CoreServices.h>
> +#include <OpenGL/gl.h>
>  //special workaround for Apple bug #6267445
>  //(OSServices Power API disabled in OSServices.h for 64bit systems)
>  #ifndef __POWER__
> @@ -160,7 +161,7 @@ static int config(uint32_t width, uint32_t height,
> uint32_t d_width, uint32_t d_
>                 image_page = 0;
>
>                 //config OpenGL View
> -               [mpGLView config:d_width:d_height:flags];
> +               [mpGLView configWidth:d_width height:d_height flags:flags];
>                 [mpGLView reshape];
>                 [[mpGLView window] setTitle:[NSString
> stringWithUTF8String:vo_wintitle ? vo_wintitle : title]];
>         }
> @@ -440,11 +441,11 @@ static int control(uint32_t request, void *data)
>         [super dealloc];
>  }
>
> -- (void) config:(uint32_t)width:(uint32_t)height:(uint32_t)flags
> +- (void) configWidth:(uint32_t)width height:(uint32_t)height
> flags:(uint32_t)flags
>  {
>         CVReturn error = kCVReturnSuccess;
>
> -       [super config:width:height:flags];
> +       [super configWidth:width height:height flags:flags];
>
>         [self releaseVideoSpecific];
>         error = CVPixelBufferCreateWithBytes(NULL, image_width,
> image_height, pixelFormat, image_datas[0], image_stride, NULL, NULL, NULL,
> &frameBuffers[0]);
> --
> 1.8.3.1 (Apple Git-46)
>

LGTM.

-- 
Xidorn Quan
GnuPG fingerprint: 6F1E DF9A D250 7505 63E2  345E 7570 8D3F 7C9A 1209


More information about the MPlayer-dev-eng mailing list