Remove assertion checking that rotations are a multiple of 90. v2.14.42
authorCarl Hetherington <cth@carlh.net>
Wed, 11 Nov 2020 22:33:31 +0000 (23:33 +0100)
committerCarl Hetherington <cth@carlh.net>
Wed, 11 Nov 2020 22:33:31 +0000 (23:33 +0100)
This is checked later, anyway (without asserting) and we have seen
files in the wild with other rotations (e.g. -135.62) which do not
appear to need rotation to be applied.

Fixes #1871.

src/lib/ffmpeg_examiner.cc

index 3b907ea54d650a6e613cf7ff663b7dc3a985c96d..c57845779d9e301c3e40f170f16e32e1188a549f 100644 (file)
@@ -177,8 +177,6 @@ FFmpegExaminer::FFmpegExaminer (shared_ptr<const FFmpegContent> c, shared_ptr<Jo
                }
 
                _rotation = *_rotation - 360 * floor (*_rotation / 360 + 0.9 / 360);
-
-               DCPOMATIC_ASSERT (fabs (*_rotation - 90 * round (*_rotation / 90)) < 2);
        }
 
        LOG_GENERAL("Temporal reference was %1", temporal_reference);