4 ms·
- Quote "$2" and "$1" - Use "$@" instead of $* - Use: OPTS=(-o "%(title)s.%(ext)s" --console-title --restrict-filenames) "${OPTS[@]}" This is what she
by JetSpiegel 7y ago
- Quote "$2" and "$1"
- Use "$@" instead of $*
- Use:
OPTS=(-o "%(title)s.%(ext)s" --console-title --restrict-filenames)
"${OPTS[@]}"
This is what shellcheck reports.
- mixmastamyk 7y agoCool, love those kind of tools. However, the youtube (and other sites id) and the format number will never contain spaces, so quoting isn't really needed here. If it gives peace of mind, sure.