[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 2/3] qobject: Add a special null QObject
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 2/3] qobject: Add a special null QObject |
Date: |
Thu, 30 Apr 2015 14:16:20 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux) |
Eric Blake <address@hidden> writes:
> From: Markus Armbruster <address@hidden>
>
> I'm going to fix the JSON parser to recognize null. The obvious
> representation of JSON null as (QObject *)NULL doesn't work, because
> the parser already uses it as an error value. Perhaps we should
> change it to free NULL for null, but that's more than I can do right
> now. Create a special null QObject instead.
Keeping my S-o-B here would clarify commit message authorship.
> The existing QDict, QList, and QString all represent something that
> is a pointer in C and could therefore be associated with NULL. But
> right now, all three of these sub-types are always non-null once
> created, so the new null sentinel object is intentionally unrelated
> to them.
>
> Signed-off-by: Markus Armbruster <address@hidden>
> Signed-off-by: Eric Blake <address@hidden>
You fixed my pasto and updated a copyright note. Thanks.