Columns: Consistency

src/columns/ColIMask.cpp

@@ -45,7 +45,6 @@ ColumnIMask::ColumnIMask ()
 void ColumnIMask::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     minimum = maximum = task.get (_name).length ();

src/columns/ColParent.cpp

@@ -45,7 +45,6 @@ ColumnParent::ColumnParent ()
 void ColumnParent::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
          if (_style == "default" || _style == "long") minimum = maximum = 36;

src/columns/ColProject.cpp

@@ -59,7 +59,6 @@ ColumnProject::ColumnProject ()
 void ColumnProject::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     std::string project = task.get (_name);

src/columns/ColRecur.cpp

@@ -68,7 +68,6 @@ void ColumnRecur::setStyle (const std::string& value)
 void ColumnRecur::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "default" ||

src/columns/ColStart.cpp

@@ -58,13 +58,14 @@ void ColumnStart::setStyle (const std::string& value)
 void ColumnStart::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "active")
       minimum = maximum = utf8_width (context.config.get ("active.indicator"));
     else
       ColumnTypeDate::measure (task, minimum, maximum);
+
+    // TODO Throw on bad format.
   }
 }
 

src/columns/ColTags.cpp

@@ -75,7 +75,6 @@ void ColumnTags::setStyle (const std::string& value)
 void ColumnTags::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "indicator")

src/columns/ColTypeDate.cpp

@@ -72,7 +72,6 @@ ColumnTypeDate::ColumnTypeDate ()
 void ColumnTypeDate::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     Datetime date (task.get_date (_name));

src/columns/ColUDA.cpp

@@ -71,7 +71,6 @@ bool ColumnUDAString::validate (const std::string& value) const
 void ColumnUDAString::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "default")
@@ -159,7 +158,6 @@ bool ColumnUDANumeric::validate (const std::string& value) const
 void ColumnUDANumeric::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "default")
@@ -239,7 +237,6 @@ bool ColumnUDADate::validate (const std::string& value) const
 void ColumnUDADate::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "default")
@@ -345,7 +342,6 @@ bool ColumnUDADuration::validate (const std::string& value) const
 void ColumnUDADuration::measure (Task& task, unsigned int& minimum, unsigned int& maximum)
 {
   minimum = maximum = 0;
-
   if (task.has (_name))
   {
     if (_style == "default")

src/columns/Column.cpp

@@ -145,7 +145,7 @@ void Column::factory (std::map <std::string, Column*>& all)
 ////////////////////////////////////////////////////////////////////////////////
 void Column::uda (std::map <std::string, Column*>& all)
 {
-  // For each UDA, instantiate and initialize ColumnUDA().
+  // For each UDA, instantiate and initialize ColumnUDA.
   std::set <std::string> udas;
 
   for (const auto& i : context.config)