[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v2 0/1] qemu-nbd: Close inherited stderr
From: |
Eric Blake |
Subject: |
Re: [PATCH v2 0/1] qemu-nbd: Close inherited stderr |
Date: |
Thu, 14 May 2020 09:55:02 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.7.0 |
On 5/14/20 9:29 AM, Eric Blake wrote:
WARNING: Block comments use a leading /* on a separate line
#20: FILE: qemu-nbd.c:919:
+ /* Remember parents stderr only if the fork option is set.
The comment could use some grammar help (s/parents/parent's/), and in
truth, I don't think it adds much beyond what the code itself is already
doing, so rather than adding another line to silence patchew, you could
instead just eliminate the comment and life would still be fine. Or if
you want a one-line comment, I might suggest:
/* Remember parent's stderr if we will restoring it. */
It helps if I don't hit 'send' too early.
/* Remember parent's stderr if we will be restoring it. */
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3226
Virtualization: qemu.org | libvirt.org