summaryrefslogtreecommitdiff
path: root/tools
diff options
context:
space:
mode:
authorZdenek Kabelac <zkabelac@redhat.com>2016-02-23 12:18:48 +0100
committerZdenek Kabelac <zkabelac@redhat.com>2016-02-23 12:25:25 +0100
commitdbc71dc05ef100a3907e6b47764b8eff71f1c7f3 (patch)
tree6381d8a0b08e9ce360a6f2d7ad9a4082693664fb /tools
parent293aabe4cd43a80fa94d7adb009fb8ab6c9d3641 (diff)
downloadlvm2-dbc71dc05ef100a3907e6b47764b8eff71f1c7f3.tar.gz
gcc: cleanup some sign warnings
When comparing unsigned with int, the comparision is made as 'unsigned' type, so make it rather explicit which type is being compared.
Diffstat (limited to 'tools')
-rw-r--r--tools/lvchange.c2
-rw-r--r--tools/lvconvert.c2
2 files changed, 2 insertions, 2 deletions
diff --git a/tools/lvchange.c b/tools/lvchange.c
index 91c872842..316a786d7 100644
--- a/tools/lvchange.c
+++ b/tools/lvchange.c
@@ -789,7 +789,7 @@ static int _lvchange_writemostly(struct logical_volume *lv)
return 0;
}
- for (s = 0; s < raid_seg->area_count; s++) {
+ for (s = 0; s < (int) raid_seg->area_count; s++) {
/*
* We don't bother checking the metadata area,
* since writemostly only affects the data areas.
diff --git a/tools/lvconvert.c b/tools/lvconvert.c
index 4e6d91b97..e7394c00d 100644
--- a/tools/lvconvert.c
+++ b/tools/lvconvert.c
@@ -90,7 +90,7 @@ struct convert_poll_id_list {
static int _lvconvert_validate_names(struct lvconvert_params *lp)
{
- int i, j;
+ unsigned i, j;
const char *names[] = {
(lp->lv_name == lp->pool_data_name) ? NULL : lp->lv_name, "converted",
lp->pool_data_name, "pool",