From d7f8a5b631cf9828bf16c78f4a5bbc0036197e30 Mon Sep 17 00:00:00 2001 From: Kamil Sroka Date: Fri, 13 Jul 2018 18:35:47 +0200 Subject: [PATCH] [ncp] remove unused variable from ncp_base_mtd.cpp (#2887) --- src/ncp/ncp_base_mtd.cpp | 15 +++++++-------- 1 file changed, 7 insertions(+), 8 deletions(-) diff --git a/src/ncp/ncp_base_mtd.cpp b/src/ncp/ncp_base_mtd.cpp index 89570e070..70e261184 100644 --- a/src/ncp/ncp_base_mtd.cpp +++ b/src/ncp/ncp_base_mtd.cpp @@ -2976,18 +2976,17 @@ void NcpBase::HandleUdpProxyStream(otMessage * aMessage, void NcpBase::HandleUdpProxyStream(otMessage *aMessage, uint16_t aPeerPort, otIp6Address &aPeerAddr, uint16_t aPort) { - otError error = OT_ERROR_NONE; uint16_t length = otMessageGetLength(aMessage); uint8_t header = SPINEL_HEADER_FLAG | SPINEL_HEADER_IID_0; - SuccessOrExit(error = mEncoder.BeginFrame(header, SPINEL_CMD_PROP_VALUE_IS, SPINEL_PROP_THREAD_UDP_PROXY_STREAM)); - SuccessOrExit(error = mEncoder.WriteUint16(length)); - SuccessOrExit(error = mEncoder.WriteMessage(aMessage)); + SuccessOrExit(mEncoder.BeginFrame(header, SPINEL_CMD_PROP_VALUE_IS, SPINEL_PROP_THREAD_UDP_PROXY_STREAM)); + SuccessOrExit(mEncoder.WriteUint16(length)); + SuccessOrExit(mEncoder.WriteMessage(aMessage)); - SuccessOrExit(error = mEncoder.WriteUint16(aPeerPort)); - SuccessOrExit(error = mEncoder.WriteIp6Address(aPeerAddr)); - SuccessOrExit(error = mEncoder.WriteUint16(aPort)); - SuccessOrExit(error = mEncoder.EndFrame()); + SuccessOrExit(mEncoder.WriteUint16(aPeerPort)); + SuccessOrExit(mEncoder.WriteIp6Address(aPeerAddr)); + SuccessOrExit(mEncoder.WriteUint16(aPort)); + SuccessOrExit(mEncoder.EndFrame()); // The `aMessage` is owned by the outbound frame and NCP buffer // after frame was finished/ended successfully. It will be freed