From: "Andreas Färber" <andreas.faerber@web.de>
To: Eric Sunshine <sunshine@sunshineco.com>
Cc: Peter Maydell <peter.maydell@linaro.org>,
Juan Pineda <juan@logician.com>,
qemu-devel@nongnu.org
Subject: Re: [Qemu-devel] [PATCH v2 2/4] softfloat: Avoid uint16 type conflict on Darwin
Date: Tue, 01 Nov 2011 17:37:02 +0100 [thread overview]
Message-ID: <4EB0202E.50701@web.de> (raw)
In-Reply-To: <47C91597-B6B1-4351-9DC3-30E405EC480B@sunshineco.com>
Am 01.11.2011 09:09, schrieb Eric Sunshine:
> Perhaps the following alternative solution would be more palatable? It's
> still tremendously ugly, but is localized to cocoa.m, thus less intrusive.
>
> -- >8 --
> Subject: [PATCH] softfloat: Avoid uint16 type conflict on Darwin
>
> cocoa.m includes <Security/cssmconfig.h> indirectly via <Cocoa/Cocoa.h>.
> cssmconfig.h defines type uint16 which unfortunately conflicts with the
> definition in qemu's softfloat.h, thus resulting in compilation failure.
> To work around the problem, #define _UINT16, which informs cssmconfig.h
> that uint16 is already defined and that it should not apply its own
> definition.
Thanks for the suggestion! _UINT16 is an interesting suggestion, however
softfloat's uint16 is not uint16_t but int, so I'd rather not do it that
way around.
(I had also decided against the AIX path of never defining uint16 and
always using system definitions, since that wouldn't work outside Cocoa
code.)
Do you have any thoughts about the include path issue? If we could keep
QEMU code from getting into #import <Cocoa/Cocoa.h> then we could
redefine the system type instead, in cocoa.m.
Andreas
next prev parent reply other threads:[~2011-11-01 16:38 UTC|newest]
Thread overview: 24+ messages / expand[flat|nested] mbox.gz Atom feed top
2011-10-31 19:17 [Qemu-devel] [PATCH v2 0/4] Cocoa patches for 1.0 Andreas Färber
2011-10-31 19:17 ` [Qemu-devel] [PATCH v2 1/4] MAINTAINERS: Add Cocoa maintainer Andreas Färber
2011-10-31 19:18 ` [Qemu-devel] [PATCH v2 2/4] softfloat: Avoid uint16 type conflict on Darwin Andreas Färber
2011-10-31 19:42 ` Peter Maydell
2011-10-31 19:17 ` Andreas Färber
2011-11-01 8:09 ` Eric Sunshine
2011-11-01 16:37 ` Andreas Färber [this message]
2011-11-01 17:59 ` [Qemu-devel] [PATCH v3 " Andreas Färber
2011-11-01 18:01 ` Peter Maydell
2011-11-01 18:05 ` Andreas Färber
2011-11-01 18:55 ` Eric Sunshine
2011-11-01 19:11 ` Andreas Färber
2011-11-01 18:17 ` Andreas Färber
2011-11-01 18:47 ` [Qemu-devel] [PATCH v2 " Eric Sunshine
2011-11-01 18:52 ` Andreas Färber
2011-11-01 19:06 ` Eric Sunshine
2011-11-01 19:25 ` Andreas Färber
2011-11-01 19:37 ` Peter Maydell
2011-11-01 19:45 ` Eric Sunshine
2011-11-01 20:21 ` Andreas Färber
2011-11-01 19:25 ` Eric Sunshine
2011-11-01 19:32 ` Andreas Färber
2011-10-31 19:18 ` [Qemu-devel] [PATCH v2 3/4] vl.c: Guard against GThread double-initialization Andreas Färber
2011-10-31 19:18 ` [Qemu-devel] [PATCH v2 4/4] cocoa: Close sheet after image file selection Andreas Färber
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=4EB0202E.50701@web.de \
--to=andreas.faerber@web.de \
--cc=juan@logician.com \
--cc=peter.maydell@linaro.org \
--cc=qemu-devel@nongnu.org \
--cc=sunshine@sunshineco.com \
/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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.