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/SyncerSet.cpp

diff --git a/parameter/SyncerSet.cpp b/parameter/SyncerSet.cpp
index 27d2c87..d427e2f 100644
--- a/parameter/SyncerSet.cpp
+++ b/parameter/SyncerSet.cpp
@@ -51,8 +51,12 @@
     _syncerSet.clear();
 }
 
-bool CSyncerSet::sync(CParameterBlackboard& parameterBlackboard, bool bBack, string& strError) const
+bool CSyncerSet::sync(CParameterBlackboard& parameterBlackboard, bool bBack, list<string>* plstrError) const
 {
+    bool bSuccess = true;
+
+    string strError;
+
     // Propagate
     SyncerSetConstIterator it;
 
@@ -62,8 +66,12 @@
 
         if (!pSyncer->sync(parameterBlackboard, bBack, strError)) {
 
-            return false;
+            if (plstrError) {
+
+                plstrError->push_back(strError);
+            }
+            bSuccess = false;
         }
     }
-    return true;
+    return bSuccess;
 }