[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 06/21] io: Fix Error usage in a comment <example>
From: |
Markus Armbruster |
Subject: |
Re: [PATCH 06/21] io: Fix Error usage in a comment <example> |
Date: |
Fri, 06 Dec 2019 08:20:44 +0100 |
User-agent: |
Gnus/5.13 (Gnus v5.13) Emacs/26.2 (gnu/linux) |
Vladimir Sementsov-Ogievskiy <address@hidden> writes:
> 30.11.2019 22:42, Markus Armbruster wrote:
>> Cc: "Daniel P. Berrangé" <address@hidden>
>> Signed-off-by: Markus Armbruster <address@hidden>
>> ---
>> include/io/task.h | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>>
>> diff --git a/include/io/task.h b/include/io/task.h
>> index 5cb9faf9f2..1abbfb8b65 100644
>> --- a/include/io/task.h
>> +++ b/include/io/task.h
>> @@ -119,7 +119,7 @@ typedef void (*QIOTaskWorker)(QIOTask *task,
>> * gboolean myobject_operation_timer(gpointer opaque)
>> * {
>> * QIOTask *task = QIO_TASK(opaque);
>> - * Error *err;*
>> + * Error *err = NULL;
>> *
>> * ...check something important...
>> * if (err) {
>>
>
> Reviewed-by: Vladimir Sementsov-Ogievskiy <address@hidden>
>
> (I also think you can safely s/Fix Error usage/Fix typo/ in subject)
I'd say it's both: the trailing * is clearly a typo, but the missing
initializer is asking Murphy for random crashes at inopportune times.
Examples better set *good* examples :)
Thanks!