Fix lack of video rotation in some cases (#2971).
authorCarl Hetherington <cth@carlh.net>
Fri, 29 Mar 2024 22:18:14 +0000 (23:18 +0100)
committerCarl Hetherington <cth@carlh.net>
Fri, 29 Mar 2024 22:18:15 +0000 (23:18 +0100)
Setting the optional to 0 here means we never even looked at the
packet side data.  This seems to be more of a problem with the
v2.17.x branch for some reason.

src/lib/ffmpeg_examiner.cc

index 15cb14ad53301605df3e96ec4398f7ffdce55b8c..787198e4a477417570880bfdfe11bc8322ccf017 100644 (file)
@@ -182,7 +182,6 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
                auto stream = _format_context->streams[*_video_stream];
                auto rotate_tag = av_dict_get (stream->metadata, "rotate", 0, 0);
                uint8_t* displaymatrix = av_stream_get_side_data (stream, AV_PKT_DATA_DISPLAYMATRIX, 0);
-               _rotation = 0;
 
                if (rotate_tag && *rotate_tag->value && strcmp(rotate_tag->value, "0")) {
                        char *tail;