]> sourceware.org Git - lvm2.git/commitdiff
lvchange: constify option array
authorZdenek Kabelac <zkabelac@redhat.com>
Sat, 27 Apr 2024 18:25:30 +0000 (20:25 +0200)
committerZdenek Kabelac <zkabelac@redhat.com>
Sun, 28 Apr 2024 22:13:43 +0000 (00:13 +0200)
tools/lvchange.c

index a6f071a91739d9a77845bbe0d3b61ca2a223023e..da4b6f191fabb5e85f33995fb91b22f851fb72b8 100644 (file)
@@ -1161,7 +1161,7 @@ static int _commit_reload(struct logical_volume *lv, uint32_t mr)
 }
 
 /* Helper: check @opt_num is listed in @opts array */
-static int _is_option_listed(int opt_enum, int *options)
+static int _is_option_listed(int opt_enum, const int *options)
 {
        int i;
 
@@ -1174,7 +1174,7 @@ static int _is_option_listed(int opt_enum, int *options)
 /* Check @opt_enum is an option allowing group commit/reload */
 static int _option_allows_group_commit(int opt_enum)
 {
-       int options[] = {
+       static const int _options[] = {
                permission_ARG,
                alloc_ARG,
                contiguous_ARG,
@@ -1197,13 +1197,13 @@ static int _option_allows_group_commit(int opt_enum)
                -1
        };
 
-       return _is_option_listed(opt_enum, options);
+       return _is_option_listed(opt_enum, _options);
 }
 
 /* Check @opt_enum requires direct commit/reload */
 static int _option_requires_direct_commit(int opt_enum)
 {
-       int options[] = {
+       static const int _options[] = {
                discards_ARG,
                zero_ARG,
                cachemode_ARG,
@@ -1213,7 +1213,7 @@ static int _option_requires_direct_commit(int opt_enum)
                -1
        };
 
-       return _is_option_listed(opt_enum, options);
+       return _is_option_listed(opt_enum, _options);
 }
 
 /*
This page took 0.034609 seconds and 5 git commands to generate.