From: Carl Hetherington <cth@carlh.net>
Date: Wed, 11 Nov 2020 22:33:31 +0000 (+0100)
Subject: Remove assertion checking that rotations are a multiple of 90.
X-Git-Tag: v2.14.42
X-Git-Url: https://git.carlh.net/gitweb/?a=commitdiff_plain;h=refs%2Ftags%2Fv2.14.42;p=dcpomatic.git

Remove assertion checking that rotations are a multiple of 90.

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.
---

diff --git a/src/lib/ffmpeg_examiner.cc b/src/lib/ffmpeg_examiner.cc
index 3b907ea54..c57845779 100644
--- a/src/lib/ffmpeg_examiner.cc
+++ b/src/lib/ffmpeg_examiner.cc
@@ -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);