Optimization: Put grep -v empty task filter inside sed.

No need to spawn off another process for this; we can do this inside the sed command that joins the task numbers with the task text.
Reformatting: Lay out _list() file numbering in multi-line.
2011-12-23 22:52:07 +01:00 · 2011-12-23 22:51:13 +01:00
1 changed files with 8 additions and 4 deletions
--- a/todo.sh
+++ b/todo.sh
@@ -775,8 +775,12 @@ _list() {
    fi
    items=$(
        sed = "$src"                                            \
-        | sed "N; s/^/     /; s/ *\(.\{$PADDING,\}\)\n/\1 /"    \
-        | grep -v "^[ 0-9]\+ *$"
+        | sed '''
+            N
+            s/^/     /
+            s/ *\([ 0-9]\{'"$PADDING"',\}\)\n/\1 /
+            /^[ 0-9]\+ *$/d
+         '''
    )
    if [ "${filter_command}" ]; then
        filtered_items=$(echo -n "$items" | eval "${filter_command}")
@@ -991,7 +995,7 @@ case $action in

    # Split multiple depri's, if comma separated change to whitespace separated
    # Loop the 'depri' function for each item
-    for item in ${*//,/ }; do
+    for item in $(echo $* | tr ',' ' '); do
        getTodo "$item"

 	if [[ "$todo" = \(?\)\ * ]]; then
@@ -1015,7 +1019,7 @@ case $action in

    # Split multiple do's, if comma separated change to whitespace separated
    # Loop the 'do' function for each item
-    for item in ${*//,/ }; do
+    for item in $(echo $* | tr ',' ' '); do
        getTodo "$item"

        # Check if this item has already been done
Author	SHA1	Message	Date
Ingo Karkat	272cd5e469	Optimization: Put grep -v empty task filter inside sed. No need to spawn off another process for this; we can do this inside the sed command that joins the task numbers with the task text.	2011-12-23 22:52:07 +01:00
Ingo Karkat	17658c852d	Reformatting: Lay out _list() file numbering in multi-line. ...like the following complex formatting and filtering steps.	2011-12-23 22:51:13 +01:00