diff options
| author | Peter Rajnoha <prajnoha@redhat.com> | 2014-11-25 14:13:04 +0100 |
|---|---|---|
| committer | Peter Rajnoha <prajnoha@redhat.com> | 2015-02-10 16:05:23 +0100 |
| commit | 2a19866a74c5101d262586ad0e08317e9a514977 (patch) | |
| tree | d7a3fe314577985443173ea0a1bd9accfdd461e4 | |
| parent | 455ef6f2f58e5af11a75eb4b369c1a48780b068d (diff) | |
| download | lvm2-2a19866a74c5101d262586ad0e08317e9a514977.tar.gz | |
cleanup: make report type condition consistent with the rest of the code
It's just more readable this way (each condition resulting in a certain
report type is always on one line), just like it's used elsewhere in the
code.
| -rw-r--r-- | tools/reporter.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/tools/reporter.c b/tools/reporter.c index 20fa4ab16..7517ac5bf 100644 --- a/tools/reporter.c +++ b/tools/reporter.c @@ -543,9 +543,8 @@ static int _report(struct cmd_context *cmd, int argc, char **argv, if ((report_type & PVSEGS) || ((report_type & (PVS | LABEL)) && (report_type & (LVS | LVSINFO | LVSSTATUS | LVSINFOSTATUS)))) report_type = PVSEGS; - else if ((report_type & LABEL) && (report_type & VGS)) - report_type = PVS; - else if (report_type & PVS) + else if ((report_type & PVS) || + ((report_type & LABEL) && (report_type & VGS))) report_type = PVS; else if (report_type & (SEGS | SEGSSTATUS)) report_type = SEGS; |