mirror of
git://sourceware.org/git/lvm2.git
synced 2025-01-02 01:18:26 +03:00
cleanup: more readable code
Simplify if() expression. Rename 'this' to 'vg'..
This commit is contained in:
parent
d8049dd17a
commit
6e71d3fbde
11
lib/cache/lvmetad.c
vendored
11
lib/cache/lvmetad.c
vendored
@ -1187,16 +1187,17 @@ struct _lvmetad_pvscan_baton {
|
|||||||
static int _lvmetad_pvscan_single(struct metadata_area *mda, void *baton)
|
static int _lvmetad_pvscan_single(struct metadata_area *mda, void *baton)
|
||||||
{
|
{
|
||||||
struct _lvmetad_pvscan_baton *b = baton;
|
struct _lvmetad_pvscan_baton *b = baton;
|
||||||
struct volume_group *this;
|
struct volume_group *vg;
|
||||||
|
|
||||||
if (!(this = mda_is_ignored(mda) ? NULL : mda->ops->vg_read(b->fid, "", mda, NULL, NULL, 1)))
|
if (mda_is_ignored(mda) ||
|
||||||
|
!(vg = mda->ops->vg_read(b->fid, "", mda, NULL, NULL, 1)))
|
||||||
return 1;
|
return 1;
|
||||||
|
|
||||||
/* FIXME Also ensure contents match etc. */
|
/* FIXME Also ensure contents match etc. */
|
||||||
if (!b->vg || this->seqno > b->vg->seqno)
|
if (!b->vg || vg->seqno > b->vg->seqno)
|
||||||
b->vg = this;
|
b->vg = vg;
|
||||||
else if (b->vg)
|
else if (b->vg)
|
||||||
release_vg(this);
|
release_vg(vg);
|
||||||
|
|
||||||
return 1;
|
return 1;
|
||||||
}
|
}
|
||||||
|
Loading…
Reference in New Issue
Block a user