commit f0c91c07f32238e351c986d2353ff548f3071316
author Patrick Williams <patrick@stwcx.xyz> Wed Nov 09 21:35:04 2016 -0600
committer Patrick Williams <patrick@stwcx.xyz> Wed Nov 09 21:35:04 2016 -0600
tree 02f7c3c7b1bacebde9ecf7c762df6ed77b6d38ce
parent 81ee91f4507fe5a75df9dfa21fc406813a09a7ff

Reduce explicit template parameters in writeData

Change-Id: I0715a93dd97d6047e85ead7e64d3dabcc314bd98
Signed-off-by: Patrick Williams <patrick@stwcx.xyz>

time-config.cpp

diff --git a/time-config.cpp b/time-config.cpp
index ff62782..9e70044 100644
--- a/time-config.cpp
+++ b/time-config.cpp
@@ -265,7 +265,7 @@
         std::cout <<"Successfully updated UseNtp=["
                   << useDhcpNtp << "]" << std::endl;
 
-        r = writeData<decltype(useDhcpNtp)>(cv_DhcpNtpFile, useDhcpNtp);
+        r = writeData(cv_DhcpNtpFile, useDhcpNtp);
     }
 
     return 0;

time-config.hpp

diff --git a/time-config.hpp b/time-config.hpp
index 0f08641..367380c 100644
--- a/time-config.hpp
+++ b/time-config.hpp
@@ -119,7 +119,7 @@
      *                      - Exception
      */
     template <typename T>
-    auto writeData(const char* fileName, const T&& data)
+    auto writeData(const char* fileName, T&& data)
     {
         std::ofstream file(fileName, std::ios::out);
         file << data;

time-manager.cpp

diff --git a/time-manager.cpp b/time-manager.cpp
index bde2e0d..206e4b6 100644
--- a/time-manager.cpp
+++ b/time-manager.cpp
@@ -188,9 +188,7 @@
         if (config.getCurrTimeOwner() == TimeConfig::timeOwners::SPLIT)
         {
             iv_HostOffset = time.getChangedOffset();
-            r = config.writeData<decltype(iv_HostOffset.count())>
-                (cv_HostOffsetFile,
-                 iv_HostOffset.count());
+            r = config.writeData(cv_HostOffsetFile, iv_HostOffset.count());
             if (r < 0)
             {
                 // probably does not make sense to crash on these..
@@ -491,9 +489,8 @@
         tmgr->updateUptimeUsec(uptimeUsec);
 
         // Persist this
-        auto r = tmgr->config.writeData<decltype(tmgr->getHostOffset().count())>
-                 (TimeManager::cv_HostOffsetFile,
-                  tmgr->getHostOffset().count());
+        auto r = tmgr->config.writeData(TimeManager::cv_HostOffsetFile,
+                                        tmgr->getHostOffset().count());
         if (r < 0)
         {
             std::cerr << "Error saving host_offset: "
@@ -510,9 +507,7 @@
 int TimeManager::resetHostOffset()
 {
     iv_HostOffset = std::chrono::microseconds(0);
-    auto r = config.writeData<decltype(iv_HostOffset.count())>
-             (cv_HostOffsetFile,
-              iv_HostOffset.count());
+    auto r = config.writeData(cv_HostOffsetFile, iv_HostOffset.count());
     config.updateSplitModeFlag(false);
     return r;
 }
@@ -865,9 +860,7 @@
     if (config.isSplitModeChanged())
     {
         iv_HostOffset = microseconds(0);
-        auto r = config.writeData<decltype(iv_HostOffset.count())>
-                 (cv_HostOffsetFile,
-                  iv_HostOffset.count());
+        auto r = config.writeData(cv_HostOffsetFile, iv_HostOffset.count());
         if (r < 0)
         {
             std::cerr <<" Error saving offset to file" << std::endl;