Unused variable SH-2034
Anti-pattern
Major
3 days ago
—
13 days old
escaped_toggle_command appears unused. Verify use (or export if used externally)
1#!/usr/bin/env bash
2# This script will toggle cowsay on or off.
3toggle_command="/usr/games/fortune"
4escaped_toggle_command=$(sed 's/[\/&]/\\&/g' <<< "$toggle_command") 5bashrc_path="/root/.bashrc"
6
7# Check if the command is already commented out in .bashrc.TORCH_CUDA_VERSION appears unused. Verify use (or export if used externally)
5
6# Install Variables
7TORCH_VERSION="2.2.2"
8TORCH_CUDA_VERSION="cu121" 9
10# Define Package URL
11FILE_URL="https://github.com/MissouriMRDT/Autonomy_Packages/raw/main/pytorch/arm64/pytorch_${TORCH_VERSION}_arm64.deb"
Description
Variables that are declared but not used for anything should be removed.
Problematic code:
foo=42
echo "$FOO"
Preferred code:
foo=42
echo "$foo"
Exceptions:
This warning may be falsely emitted when a variable is referenced indirectly, or it is intentionally not used.
- Indirection: It's the checker's intended behavior to emit this warning for any variable that is only referenced though indirection:
# foo generates a warning, even though it has five indirect references
foo=42
name=foo
echo "${!name} $((name))"
export "$name"; eval "echo $name"
declare -n name; echo "$name"
This is an intentional design decision and not a bug. If you have variables that will not have direct references, consider using an associative array in bash, or ignore this warning.
- Intentionally unused variables:
For throwaway variables, consider using
_as a dummy:
read _ last _ zip _ _ <<< "$str"
echo "$last, $zip"