From 4cc22a0bc8eb95a74bc7d77fd24ba5361650f227 Mon Sep 17 00:00:00 2001 From: Steven Moreland Date: Wed, 22 Feb 2017 03:42:31 +0000 Subject: [PATCH] Revert "Protect against null strings in hal wrapper implementation." This reverts commit 6909bfd8ee664c6ef93e1d4c1e0c4ba5650fcba9. Updated sense null strings are now allowed in hidl as of Iada25da5407c3eccdd28704d993ffdee8b15c5fe. Bug: 35421240 Change-Id: Ied1745ba7169b2e4bc0a20000503e47172538f08 --- sensors/1.0/default/convert.cpp | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/sensors/1.0/default/convert.cpp b/sensors/1.0/default/convert.cpp index 748a9639c6..306d3a3bc2 100644 --- a/sensors/1.0/default/convert.cpp +++ b/sensors/1.0/default/convert.cpp @@ -25,8 +25,8 @@ namespace V1_0 { namespace implementation { void convertFromSensor(const sensor_t &src, SensorInfo *dst) { - dst->name = src.name == nullptr ? "" : src.name; - dst->vendor = src.vendor == nullptr ? "" : src.vendor; + dst->name = src.name; + dst->vendor = src.vendor; dst->version = src.version; dst->sensorHandle = src.handle; dst->type = (SensorType)src.type; @@ -36,8 +36,8 @@ void convertFromSensor(const sensor_t &src, SensorInfo *dst) { dst->minDelay = src.minDelay; dst->fifoReservedEventCount = src.fifoReservedEventCount; dst->fifoMaxEventCount = src.fifoMaxEventCount; - dst->typeAsString = src.stringType == nullptr ? "" : src.stringType; - dst->requiredPermission = src.requiredPermission == nullptr ? "" : src.requiredPermission; + dst->typeAsString = src.stringType; + dst->requiredPermission = src.requiredPermission; dst->maxDelay = src.maxDelay; dst->flags = src.flags; }