[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 05/23] error: Improve documentation around er
From: |
Markus Armbruster |
Subject: |
Re: [Qemu-devel] [PATCH v2 05/23] error: Improve documentation around error_append_hint() |
Date: |
Fri, 18 Dec 2015 10:11:05 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/24.5 (gnu/linux) |
Eric Blake <address@hidden> writes:
> On 12/17/2015 12:00 PM, Markus Armbruster wrote:
>
>>>>>> @@ -128,6 +138,7 @@ ErrorClass error_get_class(const Error *err);
>>>>>> * If @errp is anything else, address@hidden must be NULL.
>>>>>> * The new error's class is ERROR_CLASS_GENERIC_ERROR, and its
>>>>>> * human-readable error message is made from printf-style @fmt, ...
>>>>>> + * The resulting message should not contain newlines.
>>>>>
>>>>> Should we also discourage trailing punctuation?
>>>>
>>>> Yes. How to best phrase it?
>>>
>>> Maybe:
>>>
>>> The resulting message should be a single phrase, with no newline or
>>> trailing punctuation.
>>
>> What about ending the message with an exclamation mark?
>
> Very few current users do that! An exclamation mark is still trailing
> punctuation! And I don't like shouting at users!
>
> :)
Okay, okay, I applied it %-)
- Re: [Qemu-devel] [PATCH v2 16/23] error: Strip trailing '\n' from error string arguments (again), (continued)
[Qemu-devel] [PATCH v2 07/23] qemu-nbd: Clean up "Failed to load snapshot" error message, Markus Armbruster, 2015/12/17
[Qemu-devel] [PATCH v2 12/23] error: Use error_prepend() where it makes obvious sense, Markus Armbruster, 2015/12/17
[Qemu-devel] [PATCH v2 11/23] error: Use error_reportf_err() where it makes obvious sense, Markus Armbruster, 2015/12/17
[Qemu-devel] [PATCH v2 20/23] vhdx: Fix "log that needs to be replayed" error message, Markus Armbruster, 2015/12/17
[Qemu-devel] [PATCH v2 19/23] pci-assign: Clean up "Failed to assign" error messages, Markus Armbruster, 2015/12/17