]> sourceware.org Git - lvm2.git/commitdiff
tests: get.sh declare and assign separately
authorZdenek Kabelac <zkabelac@redhat.com>
Thu, 29 Jun 2017 08:26:28 +0000 (10:26 +0200)
committerZdenek Kabelac <zkabelac@redhat.com>
Thu, 29 Jun 2017 20:23:17 +0000 (22:23 +0200)
Declare and assign separately to avoid masking return values.

Also comment-out currently unused  orig variable.

test/lib/get.sh

index 0608bc9431438cacb5e150ff5ea872bbbc292755..88144c10c5640a47d92cae8ce746a1ea1392031c 100644 (file)
@@ -28,32 +28,38 @@ trim_() {
 }
 
 pv_field() {
-       local r=$(pvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
+       local r
+       r=$(pvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
        trim_ "$r"
 }
 
 vg_field() {
-       local r=$(vgs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
+       local r
+       r=$(vgs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
        trim_ "$r"
 }
 
 lv_field() {
-       local r=$(lvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
+       local r
+       r=$(lvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
        trim_ "$r"
 }
 
 lv_first_seg_field() {
-       local r=$(lvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1" | head -1)
+       local r
+       r=$(lvs --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1" | head -1)
        trim_ "$r"
 }
 
 lvh_field() {
-       local r=$(lvs -H --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
+       local r
+       r=$(lvs -H --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
        trim_ "$r"
 }
 
 lva_field() {
-       local r=$(lvs -a --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
+       local r
+       r=$(lvs -a --config 'log{prefix=""}' --noheadings -o "$2" "${@:3}" "$1")
        trim_ "$r"
 }
 
@@ -67,8 +73,10 @@ lv_field_lv_() {
 
 lv_tree_devices_() {
        local lv="$1/$2"
-       local type=$(lv_field "$lv" segtype -a --unbuffered | head -n 1)
-       local orig=$(lv_field_lv_ "$lv" origin)
+       local type
+       type=$(lv_field "$lv" segtype -a --unbuffered | head -n 1)
+       #local orig
+       #orig=$(lv_field_lv_ "$lv" origin)
        # FIXME: should we count in also origins ?
        #test -z "$orig" || lv_tree_devices_ $1 $orig
        case "$type" in
@@ -76,7 +84,8 @@ lv_tree_devices_() {
                lv_devices "$lv"
                ;;
        mirror|raid*)
-               local log=$(lv_field_lv_ "$lv" mirror_log)
+               local log
+               log=$(lv_field_lv_ "$lv" mirror_log)
                test -z "$log" || lv_tree_devices_ "$1" "$log"
                for i in $(lv_devices "$lv")
                        do lv_tree_devices_ "$1" "$i"; done
This page took 0.035437 seconds and 5 git commands to generate.