diff options
| author | Frédéric Boisnard <fredericx.boisnard@intel.com> | 2013-07-31 17:37:20 +0200 |
|---|---|---|
| committer | David Wagner <david.wagner@intel.com> | 2014-02-12 17:04:05 +0100 |
| commit | dfbd7a605fa30c03b5e4be7e64bfd2deba64e64b (patch) | |
| tree | 7170481fd43afc9895b079cc2796d4641c5853e6 /parameter/ParameterHandle.cpp | |
| parent | 11e31aedd3af9e95a860cfc1525badd4ffd7b56f (diff) | |
| download | external_parameter-framework-dfbd7a605fa30c03b5e4be7e64bfd2deba64e64b.zip external_parameter-framework-dfbd7a605fa30c03b5e4be7e64bfd2deba64e64b.tar.gz external_parameter-framework-dfbd7a605fa30c03b5e4be7e64bfd2deba64e64b.tar.bz2 | |
Fix bug when retrieving a boolean through a handle
BZ: 127872
There is a bug in the CParameterHandle::getAsBoolean() method:
instead of accepting a reference to a bool in argument, it takes
a copy of the boolean.
The fix consists in accepting a reference as parameter.
Change-Id: Id1078534620b4cd37b81efa28471ecf8f7e644b5
Signed-off-by: Frédéric Boisnard <fredericx.boisnard@intel.com>
Reviewed-on: http://android.intel.com:8080/122811
Reviewed-by: buildbot <buildbot@intel.com>
Reviewed-by: Rocard, KevinX <kevinx.rocard@intel.com>
Reviewed-by: De Chivre, Renaud <renaud.de.chivre@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/ParameterHandle.cpp')
| -rw-r--r-- | parameter/ParameterHandle.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/parameter/ParameterHandle.cpp b/parameter/ParameterHandle.cpp index 7987ecb..9e73974 100644 --- a/parameter/ParameterHandle.cpp +++ b/parameter/ParameterHandle.cpp @@ -87,7 +87,7 @@ bool CParameterHandle::setAsBoolean(bool bValue, string& strError) return _pBaseParameter->accessAsBoolean(bValue, true, parameterAccessContext); } -bool CParameterHandle::getAsBoolean(bool bValue, string& strError) const +bool CParameterHandle::getAsBoolean(bool& bValue, string& strError) const { // Check operation validity if (!checkAccessValidity(false, 0, strError)) { |