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>

1 files changed, 11 insertions, 3 deletions

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 @@ void CSyncerSet::clear()
     _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 @@ bool CSyncerSet::sync(CParameterBlackboard& parameterBlackboard, bool bBack, str
 
         if (!pSyncer->sync(parameterBlackboard, bBack, strError)) {
 
-            return false;
+            if (plstrError) {
+
+                plstrError->push_back(strError);
+            }
+            bSuccess = false;
         }
     }
-    return true;
+    return bSuccess;
 }