diff options
| author | Zdenek Kabelac <zkabelac@redhat.com> | 2020-08-28 19:33:49 +0200 |
|---|---|---|
| committer | Zdenek Kabelac <zkabelac@redhat.com> | 2020-08-28 21:43:03 +0200 |
| commit | 19e9c88faf10bf2f9a2dd7da695b776163fb7bd4 (patch) | |
| tree | 2b59e89cc8536b086e853f2c4a4f4a8d5bd81d5d /libdm | |
| parent | ee0cb17608d2e9f6d92ea339e121713312c3f84f (diff) | |
| download | lvm2-19e9c88faf10bf2f9a2dd7da695b776163fb7bd4.tar.gz | |
gcc: do not use return with void function
Follow C norm and do not use 'return' in void function to call other
functions.
Diffstat (limited to 'libdm')
| -rw-r--r-- | libdm/dm-tools/dmsetup.c | 12 | ||||
| -rw-r--r-- | libdm/libdm-stats.c | 6 |
2 files changed, 12 insertions, 6 deletions
diff --git a/libdm/dm-tools/dmsetup.c b/libdm/dm-tools/dmsetup.c index d01b8f2e8..bf2762ae7 100644 --- a/libdm/dm-tools/dmsetup.c +++ b/libdm/dm-tools/dmsetup.c @@ -6359,13 +6359,17 @@ static void _usage(FILE *out) { switch (_base_commands[_base_command].type) { case DMSETUP_TYPE: - return _dmsetup_usage(out); + _dmsetup_usage(out); + break; case LOSETUP_TYPE: - return _losetup_usage(out); + _losetup_usage(out); + break; case STATS_TYPE: - return _stats_usage(out); + _stats_usage(out); + break; case DEVMAP_NAME_TYPE: - return _devmap_name_usage(out); + _devmap_name_usage(out); + break; } } diff --git a/libdm/libdm-stats.c b/libdm/libdm-stats.c index 05971bdad..5379bed5e 100644 --- a/libdm/libdm-stats.c +++ b/libdm/libdm-stats.c @@ -1459,8 +1459,10 @@ static void _stats_walk_start(const struct dm_stats *dms, uint64_t *flags, if (!dms->regions) return; - if (!(*flags & (DM_STATS_WALK_AREA | DM_STATS_WALK_REGION))) - return _group_walk_start(dms, flags, cur_r, cur_a, cur_g); + if (!(*flags & (DM_STATS_WALK_AREA | DM_STATS_WALK_REGION))) { + _group_walk_start(dms, flags, cur_r, cur_a, cur_g); + return; + } /* initialise cursor state */ *cur_a = 0; |