Re: [libvirt] [PATCH 1/2] util: virnetdevopenvswitch: Drop an unused variable @ovs_timeout

2018-08-08 Thread Boris Fiuczynski
On 08/08/2018 08:23 AM, Erik Skultety wrote: Technically, it was never used ever since commit @f4d06ca8fd9 introduced it, but the fact that we called VIR_FREE on it was enough for Clang to never complain about it. Signed-off-by: Erik Skultety --- CC'ing Boris, as he allowed specifying

Re: [libvirt] [PATCH 1/2] util: virnetdevopenvswitch: Drop an unused variable @ovs_timeout

2018-08-08 Thread Ján Tomko
On Wed, Aug 08, 2018 at 08:23:41AM +0200, Erik Skultety wrote: Technically, it was never used ever since commit @f4d06ca8fd9 introduced it, but the fact that we called VIR_FREE on it was enough for Clang to never complain about it. Signed-off-by: Erik Skultety --- CC'ing Boris, as he allowed

[libvirt] [PATCH 1/2] util: virnetdevopenvswitch: Drop an unused variable @ovs_timeout

2018-08-08 Thread Erik Skultety
Technically, it was never used ever since commit @f4d06ca8fd9 introduced it, but the fact that we called VIR_FREE on it was enough for Clang to never complain about it. Signed-off-by: Erik Skultety --- CC'ing Boris, as he allowed specifying timeouts for openvswitch calls, however, this