Hi,
Mika highlighted this to me for review. So here goes:
-#f1# List symlinks in detail (more detailed version of 'readlink -f' and
'whence -s')
+#f1# List symlinks in detail (more detailed version of 'readlink -f', 'whence
-s' and 'namei -l')
+#
+# Usage:
+#
[...]
The #f1# line needs to be located directly above the "sll() {" line, because
the reference-chart generation script takes a look at the line below to
determine the name of the function that is documented.
sll() {
- [[ -z "$1" ]] && printf 'Usage: %s <file(s)>\n' "$0" && return 1
- local file
- for file in "$@" ; do
+ if [[ -z ${1} ]]
+ then
I'd like this to be
if ...; then
for consistency with most of the existing code.
[...]
+ local curdir="${PWD}"
+ declare -i RTN LINODE i
+ declare -a SEENINODES
Also for consistency, "local -i RTN ..." and "local -a SEENINODES" here.
[...]
+ LINODE=$(zstat -L +inode "${file}")
+ for i in ${SEENINODES}
+ do
Also for consistency
for i in ...; do
Other than that, this looks good to me.
Thanks for your work!
Regards, Frank |