[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[PATCH V3 3/4] bash-completion:fix shellcheck SC2155-Warning
From: |
t . feng |
Subject: |
[PATCH V3 3/4] bash-completion:fix shellcheck SC2155-Warning |
Date: |
Tue, 6 Dec 2022 21:49:30 +0800 |
SC2155 (warning): Declare and assign separately to avoid masking return
values.
The exit status of the command is overridden by the exit status of the
creation of the local variable.
In grub-completion.bash.in line 115:
local config_file=$(__grub_dir)/grub.cfg
^---------^ SC2155 (warning)
In grub-completion.bash.in line 126:
local grub_dir=$(__grub_dir)
^------^ SC2155 (warning)
More: https://github.com/koalaman/shellcheck/wiki/SC2155
Signed-off-by: "t.feng" <fengtao40@huawei.com>
---
util/bash-completion.d/grub-completion.bash.in | 6 ++++--
1 file changed, 4 insertions(+), 2 deletions(-)
diff --git a/util/bash-completion.d/grub-completion.bash.in
b/util/bash-completion.d/grub-completion.bash.in
index d3399f0e1..a85f18f05 100644
--- a/util/bash-completion.d/grub-completion.bash.in
+++ b/util/bash-completion.d/grub-completion.bash.in
@@ -115,7 +115,8 @@ __grub_get_last_option () {
__grub_list_menuentries () {
local cur="${COMP_WORDS[COMP_CWORD]}"
- local config_file=$(__grub_dir)/grub.cfg
+ local config_file
+ config_file=$(__grub_dir)/grub.cfg
if [ -f "$config_file" ];then
local line IFS=$'\n'
@@ -129,7 +130,8 @@ __grub_list_menuentries () {
}
__grub_list_modules () {
- local grub_dir=$(__grub_dir)
+ local grub_dir
+ grub_dir=$(__grub_dir)
local line tmp IFS=$'\n'
COMPREPLY=()
while read -r line; do
--
2.27.0