Skip to content

Commit f0eb53e

Browse files
authored
Fix minor coverity finding (AOMediaCodec#3150)
CID 559069: Performance inefficiencies (COPY_INSTEAD_OF_MOVE) "manual_tiling_str" is passed-by-value as parameter to "operator <<", when it could be moved instead.
1 parent 2b27700 commit f0eb53e

1 file changed

Lines changed: 2 additions & 1 deletion

File tree

apps/avifgainmaputil/imageio.cc

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -101,7 +101,8 @@ void PrintEncodingSettings(const avifEncoder* encoder, bool has_gain_map) {
101101
<< ")], alpha quality ['" << encoder->qualityAlpha << "' ("
102102
<< QualityLevelString(encoder->qualityAlpha) << ")]" << gain_map_str
103103
<< ", "
104-
<< (encoder->autoTiling ? "automatic tiling" : manual_tiling_str)
104+
<< (encoder->autoTiling ? "automatic tiling"
105+
: std::move(manual_tiling_str))
105106
<< ", " << encoder->maxThreads
106107
<< " worker thread(s), please wait...\n";
107108
}

0 commit comments

Comments
 (0)