Re: [libvirt] [PATCH v3 05/28] virLockManagerLockDaemonAcquire: Drop useless check

2018-08-30 Thread John Ferlan
On 08/27/2018 04:08 AM, Michal Privoznik wrote: > The if() is completely useless since args.path is set to NULL in > the line just above. > > Signed-off-by: Michal Privoznik > --- > src/locking/lock_driver_lockd.c | 3 +-- > 1 file changed, 1 insertion(+), 2 deletions(-) > Reviewed-by: John

[libvirt] [PATCH v3 05/28] virLockManagerLockDaemonAcquire: Drop useless check

2018-08-27 Thread Michal Privoznik
The if() is completely useless since args.path is set to NULL in the line just above. Signed-off-by: Michal Privoznik --- src/locking/lock_driver_lockd.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/locking/lock_driver_lockd.c b/src/locking/lock_driver_lockd.c index