[libvirt] [PATCH] testutils: Fix coverity warning with REGENERATE_OUTPUT

2016-01-12 Thread Cole Robinson
- Don't double check for expectName - actual is always non-NULL by this point, so don't check it either --- tests/testutils.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/tests/testutils.c b/tests/testutils.c index 4ffea0c..b587f83 100644 --- a/tests/testutils.c +++

Re: [libvirt] [PATCH] testutils: Fix coverity warning with REGENERATE_OUTPUT

2016-01-12 Thread Martin Kletzander
On Tue, Jan 12, 2016 at 11:04:45AM -0500, Cole Robinson wrote: - Don't double check for expectName - actual is always non-NULL by this point, so don't check it either --- tests/testutils.c | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) ACK and safe for freeze. signature.asc

Re: [libvirt] [PATCH] testutils: Fix coverity warning with REGENERATE_OUTPUT

2016-01-12 Thread John Ferlan
On 01/12/2016 11:04 AM, Cole Robinson wrote: > - Don't double check for expectName > - actual is always non-NULL by this point, so don't check it either > --- > tests/testutils.c | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/tests/testutils.c b/tests/testutils.c

Re: [libvirt] [PATCH] testutils: Fix coverity warning with REGENERATE_OUTPUT

2016-01-12 Thread Cole Robinson
On 01/12/2016 11:22 AM, John Ferlan wrote: > > > On 01/12/2016 11:04 AM, Cole Robinson wrote: >> - Don't double check for expectName >> - actual is always non-NULL by this point, so don't check it either >> --- >> tests/testutils.c | 5 ++--- >> 1 file changed, 2 insertions(+), 3 deletions(-)