From: Peter Maydell <peter.maydell@linaro.org>
To: Gerd Hoffmann <kraxel@redhat.com>
Cc: QEMU Developers <qemu-devel@nongnu.org>
Subject: Re: [Qemu-devel] [PATCH v2 00/42] rework input handling, sdl2 support
Date: Wed, 22 Jan 2014 17:55:57 +0000 [thread overview]
Message-ID: <CAFEAcA82mDra09TvYZCoizHSewBbdGvP338gyOCJiHMkBotnNA@mail.gmail.com> (raw)
In-Reply-To: <1390407647-8659-1-git-send-email-kraxel@redhat.com>
On 22 January 2014 16:20, Gerd Hoffmann <kraxel@redhat.com> wrote:
> cocoa ui code has been updated, but is totally untested (not even
> compiled). Would be great if someone with macos hardware can give
> it a spin, and send fixup patches if needed.
OBJC ui/cocoa.o
/Users/pm215/src/qemu/ui/cocoa.m:673:17: error: use of undeclared
identifier 'button'; did you mean 'buttons'?
button |= ([event deltaY] < 0) ?
^~~~~~
buttons
/Users/pm215/src/qemu/ui/cocoa.m:496:9: note: 'buttons' declared here
int buttons = 0;
^
/Users/pm215/src/qemu/ui/cocoa.m:693:39: error: use of undeclared
identifier 'con'; did you mean 'cos'?
qemu_input_update_buttons(con, bmap, last_buttons, buttons);
^~~
cos
/usr/include/math.h:332:15: note: 'cos' declared here
extern double cos(double);
^
/Users/pm215/src/qemu/ui/cocoa.m:693:39: warning: incompatible pointer
types passing 'double (double)' to parameter of type 'QemuConsole *'
(aka 'struct QemuConsole *')
[-Wincompatible-pointer-types]
qemu_input_update_buttons(con, bmap, last_buttons, buttons);
^~~
/Users/pm215/src/qemu/include/ui/input.h:41:45: note: passing argument
to parameter 'src' here
void qemu_input_update_buttons(QemuConsole *src, uint32_t *button_map,
^
/Users/pm215/src/qemu/ui/cocoa.m:699:20: error: use of undeclared
identifier 'isMouseGrabed'; did you mean 'isMouseGrabbed'?
} else if (isMouseGrabed) {
^~~~~~~~~~~~~
isMouseGrabbed
/Users/pm215/src/qemu/ui/cocoa.m:256:10: note: 'isMouseGrabbed' declared here
BOOL isMouseGrabbed;
^
/Users/pm215/src/qemu/ui/cocoa.m:700:34: error: use of undeclared
identifier 'con'; did you mean 'cos'?
qemu_input_queue_rel(con, INPUT_AXIS_X, (int)[event deltaX]);
^~~
cos
/usr/include/math.h:332:15: note: 'cos' declared here
extern double cos(double);
^
/Users/pm215/src/qemu/ui/cocoa.m:700:34: warning: incompatible pointer
types passing 'double (double)' to parameter of type 'QemuConsole *'
(aka 'struct QemuConsole *')
[-Wincompatible-pointer-types]
qemu_input_queue_rel(con, INPUT_AXIS_X, (int)[event deltaX]);
^~~
/Users/pm215/src/qemu/include/ui/input.h:48:40: note: passing argument
to parameter 'src' here
void qemu_input_queue_rel(QemuConsole *src, InputAxis axis, int value);
^
/Users/pm215/src/qemu/ui/cocoa.m:701:34: error: use of undeclared
identifier 'con'; did you mean 'cos'?
qemu_input_queue_rel(con, INPUT_AXIS_Y, (int)[event deltaY]);
^~~
cos
/usr/include/math.h:332:15: note: 'cos' declared here
extern double cos(double);
^
/Users/pm215/src/qemu/ui/cocoa.m:701:34: warning: incompatible pointer
types passing 'double (double)' to parameter of type 'QemuConsole *'
(aka 'struct QemuConsole *')
[-Wincompatible-pointer-types]
qemu_input_queue_rel(con, INPUT_AXIS_Y, (int)[event deltaY]);
^~~
/Users/pm215/src/qemu/include/ui/input.h:48:40: note: passing argument
to parameter 'src' here
void qemu_input_queue_rel(QemuConsole *src, InputAxis axis, int value);
^
3 warnings and 5 errors generated.
make: *** [ui/cocoa.o] Error 1
thanks
-- PMM
next prev parent reply other threads:[~2014-01-22 17:56 UTC|newest]
Thread overview: 46+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-01-22 16:20 [Qemu-devel] [PATCH v2 00/42] rework input handling, sdl2 support Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 01/42] ui/sdl2 : initial port to SDL 2.0 (v2.0) Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 02/42] sdl2: baum build fix Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 03/42] sdl2: remove text console logic Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 04/42] console: export QemuConsole index, width, height Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 05/42] input: rename file to legacy Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 06/42] input: qapi: define event types Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 07/42] input: qapi: add unmapped key Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 08/42] input: qapi: add pause key Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 09/42] input: add core bits of the new input layer Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 10/42] input: keyboard: add helper functions to core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 11/42] input: keyboard: switch legacy handlers to new core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 12/42] input: keyboard: switch qmp_send_key() " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 13/42] input: keyboard: switch gtk ui " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 14/42] input: keyboard: switch sdl " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 15/42] sdl2: switch keyboard handling " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 16/42] input: keyboard: switch vnc ui " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 17/42] input: keyboard: switch spice " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 18/42] input: keyboard: switch curses " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 19/42] input: mouse: add helpers functions to core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 20/42] input: mouse: add graphic_rotate support Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 21/42] input: mouse: add qemu_input_is_absolute() Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 22/42] input: mouse: switch legacy handlers to new core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 23/42] input: mouse: switch gtk ui " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 24/42] input: mouse: switch sdl " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 25/42] sdl2: switch mouse handling " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 26/42] input: mouse: switch vnc ui " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 27/42] input: mouse: switch spice " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 28/42] input: mouse: switch monitor " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 29/42] input: keyboard: switch cocoa ui to new core [untested] Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 30/42] input: mouse: " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 31/42] input: trace events Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 32/42] input-legacy: remove kbd_put_keycode Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 33/42] input-legacy: remove kbd_mouse_has_absolute Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 34/42] input-legacy: remove kbd_mouse_is_absolute Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 35/42] input-legacy: remove kbd_mouse_event Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 36/42] input: move mouse mode notifier to new core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 37/42] input: add input_mouse_mode tracepoint Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 38/42] sdl2: simplify keymap handling Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 39/42] sdl2: codestyle fixups Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 40/42] input: move qmp_query_mice to new core Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 41/42] input: move do_mouse_set " Gerd Hoffmann
2014-01-22 16:20 ` [Qemu-devel] [PATCH v2 42/42] input: remove index_from_keycode (no users) Gerd Hoffmann
2014-01-22 17:55 ` Peter Maydell [this message]
2014-01-22 22:02 ` [Qemu-devel] [PATCH v2 00/42] rework input handling, sdl2 support Peter Maydell
2014-01-24 7:25 ` Gerd Hoffmann
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=CAFEAcA82mDra09TvYZCoizHSewBbdGvP338gyOCJiHMkBotnNA@mail.gmail.com \
--to=peter.maydell@linaro.org \
--cc=kraxel@redhat.com \
--cc=qemu-devel@nongnu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).