Diagnostics - Better parsing errors

- Improved the errors when parsing a corrupt or unrecognized pending.data
  or completed.data file.

src/T.cpp

@@ -575,6 +575,9 @@ void T::parse (const std::string& line)
                 openAttrBracket + 1, closeAttrBracket - openAttrBracket - 1);
               std::vector <std::string> pairs;
               split (pairs, attributes, ' ');
+              if (pairs.size () == 0)
+                throw std::string ("Could not find any attributes.");
+
               for (size_t i = 0; i <  pairs.size (); ++i)
               {
                 std::vector <std::string> pair;
@@ -623,15 +626,17 @@ void T::parse (const std::string& line)
 
               mDescription = line.substr (closeAnnoBracket + 2, std::string::npos);
             }
+            else
+              throw std::string ("Missing annotation brackets.");
           }
           else
-            throw std::string ("Missing attribute brackets");
+            throw std::string ("Missing attribute brackets.");
         }
         else
-          throw std::string ("Missing tag brackets");
+          throw std::string ("Missing tag brackets.");
       }
       else
-        throw std::string ("Line too short");
+        throw std::string ("Line too short.");
     }
     break;
 

src/TDB.cpp

@@ -26,6 +26,7 @@
 ////////////////////////////////////////////////////////////////////////////////
 #include <iostream>
 #include <fstream>
+#include <sstream>
 #include <sys/file.h>
 #include <unistd.h>
 #include <string.h>
@@ -104,13 +105,27 @@ bool TDB::pendingT (std::vector <T>& all)
   {
     mId = 1;
 
+    int line = 1;
     std::vector <std::string>::iterator it;
     for (it = lines.begin (); it != lines.end (); ++it)
     {
-      T t (*it);
+      try
-      t.setId (mId++);
+      {
-      if (t.getStatus () == T::pending)
+        T t (*it);
-        all.push_back (t);
+        t.setId (mId++);
+        if (t.getStatus () == T::pending)
+          all.push_back (t);
+      }
+
+      catch (std::string& e)
+      {
+        std::stringstream more;
+        more << "  Line " << line << ", in " << "pending.data";
+
+        throw e + more.str ();
+      }
+
+      ++line;
     }
 
     return true;
@@ -130,12 +145,26 @@ bool TDB::allPendingT (std::vector <T>& all)
   {
     mId = 1;
 
+    int line = 1;
     std::vector <std::string>::iterator it;
     for (it = lines.begin (); it != lines.end (); ++it)
     {
-      T t (*it);
+      try
-      t.setId (mId++);
+      {
-      all.push_back (t);
+        T t (*it);
+        t.setId (mId++);
+        all.push_back (t);
+      }
+
+      catch (std::string& e)
+      {
+        std::stringstream more;
+        more << "  Line " << line << ", in " << "pending.data";
+
+        throw e + more.str ();
+      }
+
+      ++line;
     }
 
     return true;
@@ -152,12 +181,26 @@ bool TDB::completedT (std::vector <T>& all) const
   std::vector <std::string> lines;
   if (readLockedFile (mCompletedFile, lines))
   {
+    int line = 1;
     std::vector <std::string>::iterator it;
     for (it = lines.begin (); it != lines.end (); ++it)
     {
-      T t (*it);
+      try
-      if (t.getStatus () != T::deleted)
+      {
-        all.push_back (t);
+        T t (*it);
+        if (t.getStatus () != T::deleted)
+          all.push_back (t);
+      }
+
+      catch (std::string& e)
+      {
+        std::stringstream more;
+        more << "  Line " << line << ", in " << "pending.data";
+
+        throw e + more.str ();
+      }
+
+      ++line;
     }
 
     return true;
@@ -174,11 +217,25 @@ bool TDB::allCompletedT (std::vector <T>& all) const
   std::vector <std::string> lines;
   if (readLockedFile (mCompletedFile, lines))
   {
+    int line = 1;
     std::vector <std::string>::iterator it;
     for (it = lines.begin (); it != lines.end (); ++it)
     {
-      T t (*it);
+      try
-      all.push_back (t);
+      {
+        T t (*it);
+        all.push_back (t);
+      }
+
+      catch (std::string& e)
+      {
+        std::stringstream more;
+        more << "  Line " << line << ", in " << "pending.data";
+
+        throw e + more.str ();
+      }
+
+      ++line;
     }
 
     return true;