commit b56e488f1c7fc2fd660be8ecf9ed1884efd823a7 parent b5f5b2f55cffabef87487401e38bee1842d07b9e Author: Terminator X <engineerarun@gmail.com> Date: Tue, 2 Nov 2021 10:16:17 +0530 Merge pull request #1217 from N-R-K/nuke nuke: port sxiv-rifle performance improvement over Diffstat:
| M | plugins/nuke | | | 21 | +++++++++++++-------- |
1 file changed, 13 insertions(+), 8 deletions(-)
diff --git a/plugins/nuke b/plugins/nuke @@ -259,27 +259,32 @@ handle_extension() { esac } +# sets the variable abs_target, this should be faster than calling printf abspath() { case "$1" in - /*) printf "%s\n" "$1";; - *) printf "%s\n" "$PWD/$1";; + /*) abs_target="$1";; + *) abs_target="$PWD/$1";; esac } +# storing the result to a tmp file is faster than calling listimages twice listimages() { - find -L "$(dirname "$target")" -maxdepth 1 -type f -iregex \ - '.*\(jpe?g\|bmp\|webp\|ico\|svg\|png\|gif\)$' -print0 | sort -z + find -L "///${1%/*}" -maxdepth 1 -type f -print0 | + grep -izZE '\.(jpe?g|png|gif|webp|tiff|bmp|ico|svg)$' | + sort -z | tee "$tmp" } load_dir() { - target="$(abspath "$2")" - count="$(listimages | grep -a -m 1 -ZznF "$target" | cut -d: -f1)" + abspath "$2" + tmp="${TMPDIR:-/tmp}/nuke_$$" + trap 'rm -f $tmp' EXIT + count="$(listimages "$abs_target" | grep -a -m 1 -ZznF "$abs_target" | cut -d: -f1)" if [ -n "$count" ]; then if [ "$GUI" -ne 0 ]; then - listimages | xargs -0 nohup "$1" -n "$count" -- + xargs -0 nohup "$1" -n "$count" -- < "$tmp" else - listimages | xargs -0 "$1" -n "$count" -- + xargs -0 "$1" -n "$count" -- < "$tmp" fi else shift