From 60520d62fdabbc3b722a7b9a35ee5bded4e26b50 Mon Sep 17 00:00:00 2001 From: Martin Pulec Date: Thu, 30 Nov 2023 10:15:06 +0100 Subject: [PATCH] gpujpeg enc: fixed some Coverity complains move objects instead of copy (COPY_INSTEAD_OF_MOVE) fixed CID 42421{0,3} --- src/video_compress/gpujpeg.cpp | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/src/video_compress/gpujpeg.cpp b/src/video_compress/gpujpeg.cpp index 059160b62..8ee67a6f3 100644 --- a/src/video_compress/gpujpeg.cpp +++ b/src/video_compress/gpujpeg.cpp @@ -602,7 +602,7 @@ void state_video_compress_gpujpeg::push(std::shared_ptr in_frame) } if (!m_uses_worker_threads) { - m_workers[0]->compress(in_frame); + m_workers[0]->compress(std::move(in_frame)); return; } if (!in_frame) { // pass poison pill to all workers @@ -683,7 +683,9 @@ static compress_module_info get_gpujpeg_module_info(){ std::string desc = opt.description; desc.erase(std::remove(desc.begin(), desc.end(), '\t'), desc.end()); std::replace(desc.begin(), desc.end(), '\n', ' '); - module_info.opts.emplace_back(module_option{opt.label, desc, opt.key, opt.opt_str, opt.is_boolean}); + module_info.opts.emplace_back( + module_option{ opt.label, std::move(desc), opt.key, + opt.opt_str, opt.is_boolean }); } codec codec_info;