diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2011-03-29 20:19:03 +0000 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2011-03-29 20:19:03 +0000 |
| commit | f77736cab5bafc49eee76ec1643eb615b8785329 (patch) | |
| tree | e3e23d89bdd768412312eb2365b019bb9ca164c0 /tools | |
| parent | 2d1175a8953890ee55d66f1bf0d6bebf56939916 (diff) | |
| download | lvm2-f77736cab5bafc49eee76ec1643eb615b8785329.tar.gz lvm2-f77736cab5bafc49eee76ec1643eb615b8785329.tar.xz lvm2-f77736cab5bafc49eee76ec1643eb615b8785329.zip | |
Remove double braces
Clang gives notice about possible confusion as commonly double bracces are
used when some assignment is done inside them.
Diffstat (limited to 'tools')
| -rw-r--r-- | tools/pvmove.c | 4 | ||||
| -rw-r--r-- | tools/vgreduce.c | 4 |
2 files changed, 4 insertions, 4 deletions
diff --git a/tools/pvmove.c b/tools/pvmove.c index c661e628..965a9eb7 100644 --- a/tools/pvmove.c +++ b/tools/pvmove.c @@ -121,7 +121,7 @@ static struct dm_list *_get_allocatable_pvs(struct cmd_context *cmd, int argc, } /* Remove PV if full */ - if ((pvl->pv->pe_count == pvl->pv->pe_alloc_count)) + if (pvl->pv->pe_count == pvl->pv->pe_alloc_count) dm_list_del(&pvl->list); } @@ -202,7 +202,7 @@ static struct logical_volume *_set_up_pvmove_lv(struct cmd_context *cmd, /* Find segments to be moved and set up mirrors */ dm_list_iterate_items(lvl, &vg->lvs) { lv = lvl->lv; - if ((lv == lv_mirr)) + if (lv == lv_mirr) continue; if (lv_name) { if (strcmp(lv->name, lv_name)) diff --git a/tools/vgreduce.c b/tools/vgreduce.c index 1cabc8de..5697947d 100644 --- a/tools/vgreduce.c +++ b/tools/vgreduce.c @@ -287,8 +287,8 @@ static int _make_vg_consistent(struct cmd_context *cmd, struct volume_group *vg) if (mirrored_seg->log_lv) { dm_list_iterate_items(seg, &mirrored_seg->log_lv->segments) { /* FIXME: The second test shouldn't be required */ - if ((seg->segtype == - get_segtype_from_string(vg->cmd, "error"))) { + if (seg->segtype == + get_segtype_from_string(vg->cmd, "error")) { log_print("The log device for %s/%s has failed.", vg->name, mirrored_seg->lv->name); remove_log = 1; |