move timespec2seconds exception handling to calling function

src/timer.cpp

@@ -15,6 +15,7 @@
 
 #include "comm.h"
 #include "error.h"
+#include "tokenizer.h"
 #include "fmt/chrono.h"
 
 #include <cstring>
@@ -231,9 +232,14 @@ void Timer::modify_params(int narg, char **arg)
     } else if (strcmp(arg[iarg], "timeout") == 0) {
       ++iarg;
       if (iarg < narg) {
+        try {
           _timeout = utils::timespec2seconds(arg[iarg]);
-      } else
-        error->all(FLERR, "Illegal timer command");
+        } catch (TokenizerException &) {
+          error->all(FLERR, "Illegal timeout time: {}", arg[iarg]);
+        }
+      } else {
+        utils::missing_cmd_args(FLERR, "timer timeout", error);
+      }
     } else if (strcmp(arg[iarg], "every") == 0) {
       ++iarg;
       if (iarg < narg) {

src/utils.cpp

@@ -1744,14 +1744,10 @@ double utils::timespec2seconds(const std::string &timespec)
 
   ValueTokenizer values(timespec, ":");
 
-  try {
   for (i = 0; i < 3; i++) {
     if (!values.has_next()) break;
     vals[i] = values.next_double();
   }
-  } catch (TokenizerException &) {
-    return -1.0;
-  }
 
   if (i == 3)
     return (vals[0] * 60 + vals[1]) * 60 + vals[2];