diff options
| author | Kevin Rocard <kevinx.rocard@intel.com> | 2012-12-11 16:19:17 +0100 |
|---|---|---|
| committer | David Wagner <david.wagner@intel.com> | 2014-02-12 17:03:31 +0100 |
| commit | ace81f873b910493ab884dc5a6a38ba6ec3d56d3 (patch) | |
| tree | 0835bb56ade03f1bd98834f1aaf1abeca3c00459 /parameter/SubsystemObject.cpp | |
| parent | 57096bd2de45c840090503d70f40b79565e38881 (diff) | |
| download | external_parameter-framework-ace81f873b910493ab884dc5a6a38ba6ec3d56d3.zip external_parameter-framework-ace81f873b910493ab884dc5a6a38ba6ec3d56d3.tar.gz external_parameter-framework-ace81f873b910493ab884dc5a6a38ba6ec3d56d3.tar.bz2 | |
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>
Diffstat (limited to 'parameter/SubsystemObject.cpp')
| -rw-r--r-- | parameter/SubsystemObject.cpp | 20 |
1 files changed, 18 insertions, 2 deletions
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 @@ bool CSubsystemObject::sync(CParameterBlackboard& parameterBlackboard, bool bBac if (!accessHW(true, strError)) { strError = "Unable to back synchronize configurable element " + _pInstanceConfigurableElement->getPath() + ": " + strError; + log_warning(strError); return false; } @@ -103,6 +104,7 @@ bool CSubsystemObject::sync(CParameterBlackboard& parameterBlackboard, bool bBac if (!accessHW(false, strError)) { strError = "Unable to synchronize configurable element " + _pInstanceConfigurableElement->getPath() + ": " + strError; + log_warning(strError); return false; } @@ -158,7 +160,7 @@ void CSubsystemObject::blackboardWrite(const void* pvData, uint32_t uiSize) } // 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 @@ void CSubsystemObject::log(const string& strMessage, ...) const 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 |