On 11/15/19 7:40 AM, Pino Toscano wrote: > Move the creation of a virStorageVolPtr object by lookup out of > esxStorageVolLookupByPath in an own helper. This way it can be used > also in other functions. > > Signed-off-by: Pino Toscano <ptoscano@xxxxxxxxxx> > --- > src/esx/esx_storage_backend_vmfs.c | 28 ++++++++++++++++++---------- > 1 file changed, 18 insertions(+), 10 deletions(-) > > diff --git a/src/esx/esx_storage_backend_vmfs.c b/src/esx/esx_storage_backend_vmfs.c > index 1270c21e00..ab47a4c272 100644 > --- a/src/esx/esx_storage_backend_vmfs.c > +++ b/src/esx/esx_storage_backend_vmfs.c > @@ -695,32 +695,40 @@ esxStorageVolLookupByName(virStoragePoolPtr pool, > > > > +static virStorageVolPtr > +datastorePathToStorageVol(virConnectPtr conn, const char *pool, > + const char *path) > +{ > + esxPrivate *priv = conn->privateData; > + g_autofree char *key = NULL; > + > + if (esxVI_LookupStorageVolumeKeyByDatastorePath(priv->primary, path, > + &key) < 0) { > + return NULL; > + } > + > + return virGetStorageVol(conn, pool, path, key, > + &esxStorageBackendVMFS, NULL); > +} > + > + Add an extra newline here, since this file uses triple line spacing Reviewed-by: Cole Robinson <crobinso@xxxxxxxxxx> - Cole -- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list