From 09c4fd3f55b900932caf3dc85a2ecfddca6cebb6 Mon Sep 17 00:00:00 2001 From: Jonathan Earl Brassow Date: Fri, 25 Mar 2011 21:56:28 +0000 Subject: [PATCH] cosmetic change - swapping one macro for another When I see 'seg_is_mirrored', I expect the argument to be an lv_segment. In this case, it is lvcreate_params. Both structures, have a 'segtype' entry which the macro dereferences. However, it just seems easier to understand if we do 'segtype_is_mirrored' instead. --- tools/lvcreate.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/tools/lvcreate.c b/tools/lvcreate.c index d1abdf7b0..abedf0280 100644 --- a/tools/lvcreate.c +++ b/tools/lvcreate.c @@ -346,7 +346,7 @@ static int _lvcreate_params(struct lvcreate_params *lp, lp->mirrors = 1; /* Default to 2 mirrored areas if --type mirror */ - if (seg_is_mirrored(lp)) + if (segtype_is_mirrored(lp->segtype)) lp->mirrors = 2; if (arg_count(cmd, mirrors_ARG)) {