TI-47: first call successfully creates new database but returns exit status 1

- Thanks to Georg Sauthoff.

AUTHORS

@@ -40,3 +40,4 @@ suggestions:
   hosaka
   Aaron Evers
   David Patrick
+  Georg Sauthoff

ChangeLog

@@ -6,6 +6,8 @@
           (thanks to hosaka).
 - TI-42   refresh holiday script throws an error on nb-NO locale
           (thanks to Jelle van der Waa).
+- TI-47   first call successfully creates new database but returns exit status 1
+          (thanks to Georg Sauthoff)
 - Fixed Python 3 support of the holdiay/refresh script
           (thanks to Jelle van der Waa).
 - Added missing man page link

src/commands/CmdDefault.cpp

@@ -33,8 +33,6 @@
 // Returns 0 if tracking is active, 1 if not.
 int CmdDefault (Rules& rules, Database& database)
 {
-  int status = 0;
-
   // Load the most recent interval, summarize and display.
   auto interval = getLatestInterval (database);
   if (interval.range.is_open ())
@@ -61,11 +59,9 @@ int CmdDefault (Rules& rules, Database& database)
       else
         std::cout << "There is no active time tracking.\n";
     }
-
-    status = 1;
   }
 
-  return status;
+  return 0;
 }
 
 ////////////////////////////////////////////////////////////////////////////////