gts4lv-common: audio-impl: Don't use String8::empty
With the current definition, it's an alias of default constructor. This has very different meaning from std::string empty, especially that static method String8::empty can still be called from instance context. One of such bugs are present in in ParametersUtil. Bug: 295394788 Test: make checkbuild Change-Id: I9116a0c42a6a9e94116eaaaa0fd60568db428838
This commit is contained in:
committed by
Bruno Martins
parent
ee4ea7454a
commit
523a206aa2
@@ -103,7 +103,7 @@ Result ParametersUtil::getParam(const char* name, bool* value) {
|
||||
Result retval = getParam(name, &halValue);
|
||||
*value = false;
|
||||
if (retval == Result::OK) {
|
||||
if (halValue.empty()) {
|
||||
if (halValue.length() == 0) {
|
||||
return Result::NOT_SUPPORTED;
|
||||
}
|
||||
*value = !(halValue == AudioParameter::valueOff);
|
||||
|
||||
Reference in New Issue
Block a user