[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] cocoa: Avoid warning related to multiple handle
From: |
Alexandre Raymond |
Subject: |
Re: [Qemu-devel] [PATCH] cocoa: Avoid warning related to multiple handleEvent: definitions |
Date: |
Thu, 9 Jun 2011 22:47:33 -0400 |
Tested-by: Alexandre Raymond <address@hidden>
That's definitely a better patch ;-)
Thanks,
Alexandre
On Thu, Jun 9, 2011 at 3:03 PM, Andreas Färber <address@hidden> wrote:
> Avoid compiler confusion as to which method signature to use for the
> handleEvent: selector on OSX >= 10.6 by making the variable type-safe
> as opposed to generic 'id' type.
> Requires moving the variable definition to after the class definition.
>
> ----8<----
> ui/cocoa.m: In function ‘cocoa_refresh’:
> ui/cocoa.m:997: warning: multiple methods named ‘-handleEvent:’ found
> /System/Library/Frameworks/AppKit.framework/Headers/NSTextInputContext.h:84:
> warning: using ‘-(BOOL)handleEvent:(NSEvent *)theEvent’
> ui/cocoa.m:272: warning: also found ‘-(void)handleEvent:(NSEvent *)event’
> ----8<---
>
> Reported-by: Alexandre Raymond <address@hidden>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
> ui/cocoa.m | 3 ++-
> 1 files changed, 2 insertions(+), 1 deletions(-)
>
> diff --git a/ui/cocoa.m b/ui/cocoa.m
> index 1c54759..515e684 100644
> --- a/ui/cocoa.m
> +++ b/ui/cocoa.m
> @@ -63,7 +63,6 @@ typedef struct {
> } QEMUScreen;
>
> NSWindow *normalWindow;
> -id cocoaView;
> static DisplayChangeListener *dcl;
>
> int gArgc;
> @@ -278,6 +277,8 @@ static int cocoa_keycode_to_qemu(int keycode)
> - (QEMUScreen) gscreen;
> address@hidden
>
> +QemuCocoaView *cocoaView;
> +
> address@hidden QemuCocoaView
> - (id)initWithFrame:(NSRect)frameRect
> {
> --
> 1.7.5.3
>
>
>