[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v4 02/14] qapi: Move to_c_string() to common.py
From: |
Max Reitz |
Subject: |
Re: [Qemu-devel] [PATCH v4 02/14] qapi: Move to_c_string() to common.py |
Date: |
Thu, 14 Nov 2019 10:58:21 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.1.1 |
On 14.11.19 10:20, Markus Armbruster wrote:
> Max Reitz <address@hidden> writes:
>
>> This function will be useful for code generation once we allow default
>> values, so move it to the other "C helper functions". In the process,
>> rewrite it so it supports all nonprintable and non-ASCII characters.
>>
>> Signed-off-by: Max Reitz <address@hidden>
>
> Please have a close look at commit 56a8caff92 "qapi: Restrict strings to
> printable ASCII". Do we still need the rewrite?
If that’s all that has changed, I think we will still need at least some
bits, like the " or \ escaping.
Also, actually, it looks like 56a8caff92 didn’t change the fact that
control characters are verbatim parts of the string, i.e. \u000a will
still be a literal 0xa byte, and as such must be escaped anew in the C
string.
So without having tried, I think this is still very much necessary.
> If yes: the commit message title promises code motion, but the patch is
> anything but. Adjust the title, please.
OK.
Max
signature.asc
Description: OpenPGP digital signature