From 53860dd80fd6e91ce04756032a969d4b37e00a61 Mon Sep 17 00:00:00 2001 From: Michalis Spyrou Date: Mon, 1 Jul 2019 14:20:56 +0100 Subject: COMPMID-2432 Fix defects reported by Coverity Change-Id: I887eea57cc0202ad42b76b0ea093724943116a7e Signed-off-by: Michalis Spyrou Reviewed-on: https://review.mlplatform.org/c/1446 Comments-Addressed: Arm Jenkins Tested-by: Arm Jenkins Reviewed-by: Georgios Pinitas --- src/core/ITensor.cpp | 5 +++++ src/runtime/MultiImage.cpp | 10 +++++----- src/runtime/TensorAllocator.cpp | 2 +- 3 files changed, 11 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/core/ITensor.cpp b/src/core/ITensor.cpp index 7cf04b51c4..607f5cedbf 100644 --- a/src/core/ITensor.cpp +++ b/src/core/ITensor.cpp @@ -81,6 +81,8 @@ void ITensor::print(std::ostream &s, IOFormatInfo io_fmt) const const Strides strides = this->info()->strides_in_bytes(); const PaddingSize padding = this->info()->padding(); const size_t num_channels = this->info()->num_channels(); + std::ios stream_status(nullptr); + stream_status.copyfmt(s); // Set precision if(is_data_type_float(dt) && (io_fmt.precision_type != IOFormatInfo::PrecisionType::Default)) @@ -151,6 +153,9 @@ void ITensor::print(std::ostream &s, IOFormatInfo io_fmt) const s << io_fmt.row_delim; } } + + // Restore output stream flags + s.copyfmt(stream_status); } #endif /* ARM_COMPUTE_ASSERTS_ENABLED */ diff --git a/src/runtime/MultiImage.cpp b/src/runtime/MultiImage.cpp index 6eba71bf1d..eec58d3ef4 100644 --- a/src/runtime/MultiImage.cpp +++ b/src/runtime/MultiImage.cpp @@ -1,5 +1,5 @@ /* - * Copyright (c) 2016-2018 ARM Limited. + * Copyright (c) 2016-2019 ARM Limited. * * SPDX-License-Identifier: MIT * @@ -164,7 +164,7 @@ void MultiImage::allocate() void MultiImage::create_subimage(MultiImage *image, const Coordinates &coords, unsigned int width, unsigned int height) { arm_compute::Format format = image->info()->format(); - const TensorInfo info(width, height, Format::U8); + TensorInfo info(width, height, Format::U8); switch(format) { @@ -180,21 +180,21 @@ void MultiImage::create_subimage(MultiImage *image, const Coordinates &coords, u case Format::YUYV422: case Format::UYVY422: { - const TensorInfo info_full(width, height, format); + TensorInfo info_full(width, height, format); std::get<0>(_plane).allocator()->init(*dynamic_cast(image->plane(0))->allocator(), coords, info_full); break; } case Format::NV12: case Format::NV21: { - const TensorInfo info_uv88(width / 2, height / 2, Format::UV88); + TensorInfo info_uv88(width / 2, height / 2, Format::UV88); std::get<0>(_plane).allocator()->init(*dynamic_cast(image->plane(0))->allocator(), coords, info); std::get<1>(_plane).allocator()->init(*dynamic_cast(image->plane(1))->allocator(), coords, info_uv88); break; } case Format::IYUV: { - const TensorInfo info_sub2(width / 2, height / 2, Format::U8); + TensorInfo info_sub2(width / 2, height / 2, Format::U8); std::get<0>(_plane).allocator()->init(*dynamic_cast(image->plane(0))->allocator(), coords, info); std::get<1>(_plane).allocator()->init(*dynamic_cast(image->plane(1))->allocator(), coords, info_sub2); std::get<2>(_plane).allocator()->init(*dynamic_cast(image->plane(2))->allocator(), coords, info_sub2); diff --git a/src/runtime/TensorAllocator.cpp b/src/runtime/TensorAllocator.cpp index 0612d751f0..d9616ca09d 100644 --- a/src/runtime/TensorAllocator.cpp +++ b/src/runtime/TensorAllocator.cpp @@ -105,7 +105,7 @@ TensorAllocator &TensorAllocator::operator=(TensorAllocator &&o) noexcept return *this; } -void TensorAllocator::init(const TensorAllocator &allocator, const Coordinates &coords, TensorInfo sub_info) +void TensorAllocator::init(const TensorAllocator &allocator, const Coordinates &coords, TensorInfo &sub_info) { // Get parent info const TensorInfo parent_info = allocator.info(); -- cgit v1.2.1