Re: [PATCH] Introduce QEMU_NEW()

Alexander Graf
2011-07-25T09:43:09Z

On 25.07.2011, at 11:37, Sasha Levin wrote:

> On Mon, 2011-07-25 at 11:32 +0200, Alexander Graf wrote:
>> On 25.07.2011, at 10:51, Avi Kivity wrote:
>>
>>> qemu_malloc() is type-unsafe as it returns a void pointer. Introduce
>>> QEMU_NEW() (and QEMU_NEWZ()), which return the correct type.
>>
>> What does this buy you over
>>
>> type *x = qemu_malloc(sizeof(type));
>>
>> ? I find the non-C++ version easier to read even.
>
> It'll warn when you do silly things such as:
>
> struct some_struct *k;
>
> k = qemu_malloc(sizeof(k));

Hm - is there any way to get this without adding upper case C++'ish macros?

Alex