splitmuxsink: Fix if condition in drop-frame timecode wrap-around
authorVivia Nikolaidou <vivia@ahiru.eu>
Thu, 11 Oct 2018 10:55:01 +0000 (13:55 +0300)
committerVivia Nikolaidou <vivia@ahiru.eu>
Thu, 11 Oct 2018 10:58:34 +0000 (13:58 +0300)
Was previously: if ( x | y && a == b). Changed it into if ((x & y) && (a
== b)).

gst/multifile/gstsplitmuxsink.c

index bd9ed80..c8969f7 100644 (file)
@@ -1199,8 +1199,8 @@ calculate_next_max_timecode (GstSplitMuxSink * splitmux,
         day_in_ns - cur_tc_time + target_tc_time +
         splitmux->fragment_start_time;
 
-    if (cur_tc->config.flags | GST_VIDEO_TIME_CODE_FLAGS_DROP_FRAME &&
-        cur_tc->config.fps_d == 1001) {
+    if ((cur_tc->config.flags & GST_VIDEO_TIME_CODE_FLAGS_DROP_FRAME) &&
+        (cur_tc->config.fps_d == 1001)) {
       /* Checking fps_d is probably unneeded, but better safe than sorry
        * (e.g. someone accidentally set a flag) */
       guint frames_of_daily_jam;