Re: [libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-09 Thread Matthias Bolte
2012/8/6 Michal Privoznik mpriv...@redhat.com: On 05.08.2012 13:48, Matthias Bolte wrote: esxVI_LookupHostSystemProperties guarantees that hostSystem is non-NULL. Remove redundant NULL checks from callers. Also prefer esxVI_GetStringValue over open-coding the logic. ---

Re: [libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-06 Thread Michal Privoznik
On 05.08.2012 13:48, Matthias Bolte wrote: esxVI_LookupHostSystemProperties guarantees that hostSystem is non-NULL. Remove redundant NULL checks from callers. Also prefer esxVI_GetStringValue over open-coding the logic. --- src/esx/esx_driver.c | 82

[libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-05 Thread Matthias Bolte
esxVI_LookupHostSystemProperties guarantees that hostSystem is non-NULL. Remove redundant NULL checks from callers. Also prefer esxVI_GetStringValue over open-coding the logic. --- src/esx/esx_driver.c | 82 +--- src/esx/esx_vi.c |4 +-

Re: [libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result

2012-08-05 Thread Ata Bohra
Date: Sun, 5 Aug 2012 13:48:01 +0200 From: matthias.bo...@googlemail.com To: libvir-list@redhat.com Subject: [libvirt] [PATCH] esx: Remove redundant checks for esxVI_LookupHostSystemProperties result esxVI_LookupHostSystemProperties guarantees that hostSystem is non-NULL. Remove