deinterlace: Remove incorrect logic
authorRobert Swain <robert.swain@collabora.co.uk>
Tue, 18 Sep 2012 22:39:01 +0000 (00:39 +0200)
committerRobert Swain <robert.swain@collabora.co.uk>
Tue, 18 Sep 2012 22:39:01 +0000 (00:39 +0200)
I don't understand why these lines were added, they don't make sense to
me now and both David and I agree that removing them moves closer to
related logic being correct, therefore, they're being removed.

I've tested a few progressive, interlaced and telecine clips and they
all behave properly timestamp-wise and visually after these changes.

gst/deinterlace/gstdeinterlace.c

index e73d087..ed9d96e 100644 (file)
@@ -1838,10 +1838,6 @@ restart:
   if (self->cur_field_idx < 0)
     return ret;
 
-  if (!flushing && self->cur_field_idx < 1) {
-    return ret;
-  }
-
   /* deinterlace bottom_field */
   if ((self->field_history[self->cur_field_idx].flags ==
           PICTURE_INTERLACED_BOTTOM && (self->fields == GST_DEINTERLACE_BF