[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH v2 3/7] block/sheepdog: remove spurious NULL che
From: |
Eric Blake |
Subject: |
Re: [Qemu-devel] [PATCH v2 3/7] block/sheepdog: remove spurious NULL check |
Date: |
Wed, 30 Aug 2017 14:42:51 -0500 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0 |
On 08/30/2017 11:57 AM, Jeff Cody wrote:
> 'tag' is already checked in the lines immediately preceding this check,
> and set to non-NULL if NULL. No need to check again, it hasn't changed.
>
> Signed-off-by: Jeff Cody <address@hidden>
> ---
> block/sheepdog.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
Reviewed-by: Eric Blake <address@hidden>
--
Eric Blake, Principal Software Engineer
Red Hat, Inc. +1-919-301-3266
Virtualization: qemu.org | libvirt.org
signature.asc
Description: OpenPGP digital signature
- Re: [Qemu-devel] [PATCH v2 1/7] block/ssh: don't call libssh2_init() in block_init(), (continued)
[Qemu-devel] [PATCH v2 6/7] block/curl: fix minor memory leaks, Jeff Cody, 2017/08/30
[Qemu-devel] [PATCH v2 2/7] block/ssh: make compliant with coding guidelines, Jeff Cody, 2017/08/30
[Qemu-devel] [PATCH v2 5/7] block/curl: check error return of curl_global_init(), Jeff Cody, 2017/08/30
[Qemu-devel] [PATCH v2 3/7] block/sheepdog: remove spurious NULL check, Jeff Cody, 2017/08/30
[Qemu-devel] [PATCH v2 4/7] block/sheepdog: code beautification, Jeff Cody, 2017/08/30
[Qemu-devel] [PATCH v2 7/7] block/curl: code cleanup to comply with coding style, Jeff Cody, 2017/08/30