From 1365c024d315ebd2663e2b5c8303ae7447e9e85c Mon Sep 17 00:00:00 2001
From: Scott Worley <scottworley@scottworley.com>
Date: Sat, 8 Mar 2014 22:49:03 -0800
Subject: [PATCH] Be verbose about skipping work

Also, move the exposure calculation inside need-to-do-anything?
conditional, reducing the do-nothing case from two forks to one.
---
 pics-preprocess | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/pics-preprocess b/pics-preprocess
index 8b8dcb4..6563cab 100755
--- a/pics-preprocess
+++ b/pics-preprocess
@@ -13,16 +13,22 @@ for f;do
       mkdir -p {,sm/}"$level"
       base="$(basename "$f")"
       out="$level/${base%.NEF}.jpeg"
-      exposure=$(dc <<< "2k $level 4 / 3.5-p")  # 10 to 30 -> -1 to 4 in .25 steps
-      if [[ ! -s "$out" ]];then
+      if [[ -s "$out" ]];then
+        echo "Skipping $out"
+      else
+        exposure=$(dc <<< "2k $level 4 / 3.5-p")  # 10 to 30 -> -1 to 4 in .25 steps
         pics-run-ufraw "$exposure" "$f" "$out"
       fi
-      if [[ ! -s "sm/$out" ]];then
+      if [[ -s "sm/$out" ]];then
+        echo "Skipping sm/$out"
+      else
         convert "$out" -geometry "x$SMALLHEIGHT" "sm/$out"
       fi
     done
   else
-    if [[ ! -s "sm/$f" ]];then
+    if [[ -s "sm/$f" ]];then
+      echo "Skipping sm/$f"
+    else
       convert "$f" -geometry "x$SMALLHEIGHT" "sm/$f"
     fi
   fi
-- 
2.44.1