Remove assertion checking that rotations are a multiple of 90.
authorCarl Hetherington <cth@carlh.net>
Wed, 11 Nov 2020 22:35:32 +0000 (23:35 +0100)
committerCarl Hetherington <cth@carlh.net>
Wed, 11 Nov 2020 22:35:32 +0000 (23:35 +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.

Backported-from-commit: f235ed07f969e1b8b9d3d05082dcdd68ceae771e
Backported-from-branch: master

src/lib/ffmpeg_examiner.cc

index 5683555e4aaffc42b3bce4493b17e9a5f1f3a72b..8a26f56260bb3571b51a7acbb8a7e76dc02ccf8e 100644 (file)
@@ -184,8 +184,6 @@ DCPOMATIC_ENABLE_WARNINGS
                }
 
                _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);