Implement feedback from CodeReview.SE

Issue #2 resolved
Michael Diamond
repo owner created an issue

See https://codereview.stackexchange.com/q/170958/14237

  • Multi-tiered cache (Issue #1)
  • Consider disown-ing background tasks - not sure if this is necessary or beneficial
  • Avoid cat in favor of using Bash to read (while...do; ...; done <$file)
  • Consider avoiding a mktemp'ed directory, possibly encoding $USER and/or $$ in the path
  • Consider sqlite or some other backing store (likely not in V1)
  • Avoid find where feasible (already in progress in 76e730b9)
  • Consider alternative to current heredoc syntax, such as using 'single-quotes' and breaking out as needed ('$foo'"$bar"'$baz' -> $foo...$baz)
  • No need for in "$@" in for loops

Comments (2)

  1. Log in to comment