Fix PlayerVideo::has_j2k to cope with subtitles and fades.

[dcpomatic.git] / src / lib / player_video.cc

diff --git a/src/lib/player_video.cc b/src/lib/player_video.cc
index 765896a0eb532d9c314143b6ce6929408bf82b85..2b1e85fc95cb2fd33af6b96b7875b496daa17137 100644 (file)
--- a/src/lib/player_video.cc
+++ b/src/lib/player_video.cc
@@ -167,14 +167,14 @@ PlayerVideo::send_binary (shared_ptr<Socket> socket) const
 bool
 PlayerVideo::has_j2k () const
 {
-       /* XXX: burnt-in subtitle; maybe other things */
+       /* XXX: maybe other things */
 
        shared_ptr<const J2KImageProxy> j2k = dynamic_pointer_cast<const J2KImageProxy> (_in);
        if (!j2k) {
                return false;
        }
 
-       return _crop == Crop () && _inter_size == j2k->size();
+       return _crop == Crop () && _inter_size == j2k->size() && !_subtitle && !_fade;
 }
 
 Data