BTS

Message4970

Author ft
Recipients Xk2c
Date 2015-05-24.09:29:31
Content
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
History
Date User Action Args
2015-05-24 09:29:31ftsetrecipients: + Xk2c
2015-05-24 09:29:31ftsetmessageid: <1432459771.62.0.210244649158.issue1351@bts.grml.org>
2015-05-24 09:29:31ftlinkissue1351 messages
2015-05-24 09:29:31ftcreate