[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH] cocoa: Provide central qemu_main() prototype
From: |
Alexandre Raymond |
Subject: |
Re: [Qemu-devel] [PATCH] cocoa: Provide central qemu_main() prototype |
Date: |
Sun, 29 May 2011 19:53:29 -0400 |
Hi Andreas,
Just thinking out loud here : wouldn't it make more sense to put the
main() of each gui framework directly in its corresponding file and
select the right one in the makefile using the configure options?
so you'd have
-no gui -> ui/no_gui.c:main() -> qemu_main() === compile with no_gui.c + vl.c
-sdl -> ui/sdl.c:main() -> qemu_main() === compile with sdl.c + vl.c
-cocoa -> ui/cocoa.m:main() -> qemu_main() === compile with cocoa.m + vl.c
with ui/no_gui.c, ui/sdl.c and ui/cocoa.m each having their own main():
----8<----
...
int main(...) {
return qemu_main(....);
}
...
----8<----
and definitively rename main() to qemu_main() in vl.c ?
Alexandre
On Sun, May 29, 2011 at 3:58 PM, Andreas Färber <address@hidden> wrote:
> This fixes a missing prototype warning in vl.c and obsoletes
> the prototype in cocoa.m. Adjust callers in cocoa.m to supply
> third argument, which is currently only used on Linux/ppc.
>
> The prototype is designed so that it could be shared with SDL
> and other frontends, if desired.
>
> Cc: Alexandre Raymond <address@hidden>
> Signed-off-by: Andreas Färber <address@hidden>
> ---
> qemu-common.h | 5 +++++
> ui/cocoa.m | 6 +++---
> 2 files changed, 8 insertions(+), 3 deletions(-)
>
> diff --git a/qemu-common.h b/qemu-common.h
> index b851b20..218289c 100644
> --- a/qemu-common.h
> +++ b/qemu-common.h
> @@ -131,6 +131,11 @@ static inline char *realpath(const char *path, char
> *resolved_path)
>
> #endif /* !defined(NEED_CPU_H) */
>
> +/* main function, renamed */
> +#if defined(CONFIG_COCOA)
> +int qemu_main(int argc, char **argv, char **envp);
> +#endif
> +
> /* bottom halves */
> typedef void QEMUBHFunc(void *opaque);
>
> diff --git a/ui/cocoa.m b/ui/cocoa.m
> index 1ff1ac6..6566e46 100644
> --- a/ui/cocoa.m
> +++ b/ui/cocoa.m
> @@ -23,6 +23,7 @@
> */
>
> #import <Cocoa/Cocoa.h>
> +#include <crt_externs.h>
>
> #include "qemu-common.h"
> #include "console.h"
> @@ -61,7 +62,6 @@ typedef struct {
> int bitsPerPixel;
> } QEMUScreen;
>
> -int qemu_main(int argc, char **argv); // main defined in qemu/vl.c
> NSWindow *normalWindow;
> id cocoaView;
> static DisplayChangeListener *dcl;
> @@ -794,7 +794,7 @@ static int cocoa_keycode_to_qemu(int keycode)
> COCOA_DEBUG("QemuCocoaAppController: startEmulationWithArgc\n");
>
> int status;
> - status = qemu_main(argc, argv);
> + status = qemu_main(argc, argv, *_NSGetEnviron());
> exit(status);
> }
>
> @@ -876,7 +876,7 @@ int main (int argc, const char * argv[]) {
> !strcmp(opt, "-nographic") ||
> !strcmp(opt, "-version") ||
> !strcmp(opt, "-curses")) {
> - return qemu_main(gArgc, gArgv);
> + return qemu_main(gArgc, gArgv, *_NSGetEnviron());
> }
> }
> }
> --
> 1.7.5.3
>
>
- Re: [Qemu-devel] [PATCH 1/6] Fix incorrect check for fdatasync() in configure, (continued)
[Qemu-devel] [PATCH 2/6] Cocoa: avoid displaying window when command-line contains '-h', Alexandre Raymond, 2011/05/27
[Qemu-devel] [PATCH 3/6] Fix compilation warning due to incorrectly specified type, Alexandre Raymond, 2011/05/27
[Qemu-devel] [PATCH 4/6] Fix missing prototype under cocoa for qemu_main, Alexandre Raymond, 2011/05/27
[Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Alexandre Raymond, 2011/05/27
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Stefan Weil, 2011/05/27
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Markus Armbruster, 2011/05/27
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Stefan Weil, 2011/05/27
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Alexandre Raymond, 2011/05/27
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Paolo Bonzini, 2011/05/28
- Re: [Qemu-devel] [PATCH 5/6] Remove warning in printf due to type mismatch, Stefan Weil, 2011/05/28
[Qemu-devel] [PATCH 6/6] Avoid compilation warning regarding kvm under darwin, Alexandre Raymond, 2011/05/27
Re: [Qemu-devel] [PATCH 0/6] Fix compilation issues under darwin, Andreas Färber, 2011/05/28