From e57380151749404424982012d0d3bfdf2858d7e0 Mon Sep 17 00:00:00 2001
From: Paul Beckingham <paul@beckingham.net>
Date: Mon, 22 Aug 2011 23:04:38 -0400
Subject: [PATCH] TDB2 - Diagnostics

- Added a diagnostic dump of the TDB2 state in debug mode.
- Added a call to TDB2::dump at the start of TDB2::commit.
- Removed unnecessary calls to TDB2::dump.
---
 src/Context.cpp |   1 -
 src/TDB2.cpp    | 106 +++++++++++++++++++++++++++---------------------
 src/TDB2.h      |   3 +-
 3 files changed, 62 insertions(+), 48 deletions(-)

diff --git a/src/Context.cpp b/src/Context.cpp
index 1380e7b35..2018c30e4 100644
--- a/src/Context.cpp
+++ b/src/Context.cpp
@@ -153,7 +153,6 @@ int Context::initialize (int argc, const char** argv)
     tdb.clear ();                   // TODO Obsolete
     tdb.location (data_dir);        // TODO Obsolete
     tdb2.set_location (data_dir);
-    tdb2.dump ();
 
     // Hook system init, plus post-start event occurring at the first possible
     // moment after hook initialization.
diff --git a/src/TDB2.cpp b/src/TDB2.cpp
index c76dc3ff6..024de921a 100644
--- a/src/TDB2.cpp
+++ b/src/TDB2.cpp
@@ -27,6 +27,7 @@
 
 //#include <iostream> // TODO Remove.
 #include <Context.h>
+#include <Color.h>
 #include <text.h>
 #include <TDB2.h>
 
@@ -341,6 +342,59 @@ int TF2::id (const std::string& uuid)
 }
 
 ////////////////////////////////////////////////////////////////////////////////
+// <label> <rw><dirty> <tasks> <lines> <contents>
+//
+// label:    <label %+14s>
+// rw:       <rw>
+// dirty:    <!|->
+// tasks:    T<tasks %04d>+<added %03d>~<changed %03d>
+// lines:    L<lines %04d>+<added %03d>
+// contents: C<bytes %06d>
+const std::string TF2::dump ()
+{
+  Color red    ("rgb500 on rgb100");
+  Color yellow ("rgb550 on rgb220");
+  Color green  ("rgb050 on rgb010");
+
+  // File label.
+  std::string label;
+  std::string::size_type slash = _file.data.rfind ('/');
+  if (slash != std::string::npos)
+    label = rightJustify (_file.data.substr (slash + 1), 14);
+
+  // File mode.
+  std::string mode = std::string (_file.readable () ? "r" : "-") +
+                     std::string (_file.writable () ? "w" : "-");
+       if (mode == "r-") mode = red.colorize (mode);
+  else if (mode == "rw") mode = green.colorize (mode);
+  else                   mode = yellow.colorize (mode);
+
+  // Hygiene.
+  std::string hygiene = _dirty ? red.colorize ("O") : green.colorize ("-");
+
+  std::string tasks          = green.colorize  (rightJustifyZero ((int) _tasks.size (),          4));
+  std::string tasks_added    = red.colorize    (rightJustifyZero ((int) _added_tasks.size (),    3));
+  std::string tasks_modified = yellow.colorize (rightJustifyZero ((int) _modified_tasks.size (), 3));
+  std::string lines          = green.colorize  (rightJustifyZero ((int) _lines.size (),          4));
+  std::string lines_added    = red.colorize    (rightJustifyZero ((int) _added_lines.size (),    3));
+  std::string contents       = green.colorize  (rightJustifyZero ((int) _contents.size (),       6));
+
+  char buffer[256];  // Composed string is actually 246 bytes.  Yikes.
+  snprintf (buffer, 256, "%14s %s %s T%s+%s~%s L%s+%s C%s",
+            label.c_str (),
+            mode.c_str (),
+            hygiene.c_str (),
+            tasks.c_str (),
+            tasks_added.c_str (),
+            tasks_modified.c_str (),
+            lines.c_str (),
+            lines_added.c_str (),
+            contents.c_str ());
+
+  return std::string (buffer);
+}
+
+////////////////////////////////////////////////////////////////////////////////
 
 
 
@@ -419,7 +473,6 @@ void TDB2::modify (const Task& task)
 void TDB2::commit ()
 {
   dump ();
-
   context.timer_gc.start ();
 
   pending.commit ();
@@ -429,8 +482,6 @@ void TDB2::commit ()
   synch_key.commit ();
 
   context.timer_gc.stop ();
-
-  dump ();
 }
 
 ////////////////////////////////////////////////////////////////////////////////
@@ -501,54 +552,17 @@ bool TDB2::verifyUniqueUUID (const std::string& uuid)
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-//
-//  File           RW State Tasks + - ~ lines + - Bytes
-//  -------------- -- ----- ----- - - - ----- - - -----
-//  pending.data   rw clean   123t  +2t  -1t ~1t
-//  completed.data rw clean   123t  +2t      ~1t
-//  undo.data      rw clean   123t  +2t      ~1t
-//  backlog.data   rw clean   123t  +2t      ~1t
-//  synch-key.data rw clean                        123b
-//
 void TDB2::dump ()
 {
   if (context.config.getBoolean ("debug"))
   {
-    ViewText view;
-    view.width (context.getWidth ());
-    view.add (Column::factory ("string",       "File"));
-    view.add (Column::factory ("string.right", "RW"));
-    view.add (Column::factory ("string.right", "State"));
-    view.add (Column::factory ("string.right", "Tasks"));
-    view.add (Column::factory ("string.right", "+"));
-    view.add (Column::factory ("string.right", "~"));
-    view.add (Column::factory ("string.right", "Lines"));
-    view.add (Column::factory ("string.right", "+"));
-    view.add (Column::factory ("string.right", "Bytes"));
-
-    dump_file (view, "pending.data", pending);
-    dump_file (view, "completed.data", completed);
-    dump_file (view, "undo.data", undo);
-    dump_file (view, "backlog.data", backlog);
-    dump_file (view, "synch_key.data", synch_key);
-    context.debug (view.render ());
+    context.debug (pending.dump ());
+    context.debug (completed.dump ());
+    context.debug (undo.dump ());
+    context.debug (backlog.dump ());
+    context.debug (synch_key.dump ());
+    context.debug ("");
   }
 }
 
 ////////////////////////////////////////////////////////////////////////////////
-void TDB2::dump_file (ViewText& view, const std::string& label, TF2& tf)
-{
-  int row = view.addRow ();
-  view.set (row, 0, label);
-  view.set (row, 1, std::string (tf._file.readable () ? "r" : "-") +
-                    std::string (tf._file.writable () ? "w" : "-"));
-  view.set (row, 2, tf._dirty ? "dirty" : "clean");
-  view.set (row, 3, tf._loaded_tasks ? (format ((int)tf._tasks.size ())) : "-");
-  view.set (row, 4, (int)tf._added_tasks.size ());
-  view.set (row, 5, (int)tf._modified_tasks.size ());
-  view.set (row, 6, tf._loaded_lines ? (format ((int)tf._lines.size ())) : "-");
-  view.set (row, 7, (int)tf._added_lines.size ());
-  view.set (row, 8, tf._loaded_contents ? (format ((int)tf._contents.size ())) : "-");
-}
-
-////////////////////////////////////////////////////////////////////////////////
diff --git a/src/TDB2.h b/src/TDB2.h
index 7d642da57..86dee9431 100644
--- a/src/TDB2.h
+++ b/src/TDB2.h
@@ -62,6 +62,8 @@ public:
   std::string uuid (int);
   int id (const std::string&);
 
+  const std::string dump ();
+
 public:
   bool _read_only;
   bool _dirty;
@@ -97,7 +99,6 @@ public:
   int  next_id ();
 
   void dump ();
-  void dump_file (ViewText&, const std::string&, TF2&);
 
 private:
   bool verifyUniqueUUID (const std::string&);