commit ace81f873b910493ab884dc5a6a38ba6ec3d56d3
author Kevin Rocard <kevinx.rocard@intel.com> Tue Dec 11 16:19:17 2012 +0100
committer David Wagner <david.wagner@intel.com> Wed Feb 12 17:03:31 2014 +0100
tree 0835bb56ade03f1bd98834f1aaf1abeca3c00459
parent 57096bd2de45c840090503d70f40b79565e38881

PFW: Do not stop on hard sync errors

BZ: 76263

When synchronization fails, the parameter-framework should
log a warning and continue synchronization instead of stopping

Change-Id: Ic12d8798ea25584db714ee26e644fac793c28881
Signed-off-by: Kevin Rocard <kevinx.rocard@intel.com>
Reviewed-on: http://android.intel.com:8080/81825
Reviewed-by: Centelles, Sylvain <sylvain.centelles@intel.com>
Reviewed-by: Denneulin, Guillaume <guillaume.denneulin@intel.com>
Tested-by: Dixon, CharlesX <charlesx.dixon@intel.com>
Reviewed-by: cactus <cactus@intel.com>
Tested-by: cactus <cactus@intel.com>

parameter/SubsystemObject.cpp

diff --git a/parameter/SubsystemObject.cpp b/parameter/SubsystemObject.cpp
index f98509e..8c5c5e4 100644
--- a/parameter/SubsystemObject.cpp
+++ b/parameter/SubsystemObject.cpp
@@ -93,6 +93,7 @@
         if (!accessHW(true, strError)) {
 
             strError = "Unable to back synchronize configurable element " + _pInstanceConfigurableElement->getPath() + ": " + strError;
+            log_warning(strError);
 
             return false;
         }
@@ -103,6 +104,7 @@
         if (!accessHW(false, strError)) {
 
             strError = "Unable to synchronize configurable element " + _pInstanceConfigurableElement->getPath() + ": " + strError;
+            log_warning(strError);
 
             return false;
         }
@@ -158,7 +160,7 @@
 }
 
 // Logging
-void CSubsystemObject::log(const string& strMessage, ...) const
+void CSubsystemObject::log_info(const string& strMessage, ...) const
 {
     char acBuffer[512];
     va_list listPointer;
@@ -169,7 +171,21 @@
 
     va_end(listPointer);
 
-    _pInstanceConfigurableElement->log(acBuffer);
+    _pInstanceConfigurableElement->log_info(acBuffer);
+}
+
+void CSubsystemObject::log_warning(const string& strMessage, ...) const
+{
+    char acBuffer[512];
+    va_list listPointer;
+
+    va_start(listPointer, strMessage);
+
+    vsnprintf(acBuffer, sizeof(acBuffer), strMessage.c_str(), listPointer);
+
+    va_end(listPointer);
+
+    _pInstanceConfigurableElement->log_warning(acBuffer);
 }
 
 // Amendment