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

diff --git a/parameter/SelectionCriterion.cpp b/parameter/SelectionCriterion.cpp
index 453f244..77bca4d 100644
--- a/parameter/SelectionCriterion.cpp
+++ b/parameter/SelectionCriterion.cpp
@@ -55,13 +55,13 @@
 
         _iState = iState;
 
-        log("Selection criterion changed event: %s", getFormattedDescription(false).c_str());
+        log_info("Selection criterion changed event: %s", getFormattedDescription(false).c_str());
 
         // Check if the previous criterion value has been taken into account (i.e. at least one Configuration was applied
         // since the last criterion change)
-        if (_uiNbModifications > 0) {
+        if (_uiNbModifications != 0) {
 
-            log("Warning: Selection criterion \"%s\" has been modified %d time(s) without any configuration application", getName().c_str(), _uiNbModifications);
+            log_warning("Selection criterion \"%s\" has been modified %d time(s) without any configuration application", getName().c_str(), _uiNbModifications);
         }
 
         // Track the number of modifications for this criterion