[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report
From: |
Markus Armbruster |
Subject: |
Re: [PATCH v2 02/23] qapi/misc-target: Wrap long 'SEV Attestation Report' long lines |
Date: |
Mon, 30 Aug 2021 17:08:18 +0200 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/27.2 (gnu/linux) |
Markus Armbruster <armbru@redhat.com> writes:
> Philippe Mathieu-Daudé <philmd@redhat.com> writes:
>
>> Wrap long lines before 70 characters for legibility.
>>
>> Suggested-by: Markus Armbruster <armbru@redhat.com>
>> Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
>
> Reviewed-by: Markus Armbruster <armbru@redhat.com>
With the typo spotted by Dave fixed, of course.
>
>> ---
>> Note: it would be nice if checkpatch enforce 70 char for json
>> (or at least QAPI json), that would save future
>> developer modifying QAPI definitions reformating time.
>
> Yes.