Revert "codecparsers: remove ignored increment of return"
authorTim-Philipp Müller <tim@centricular.com>
Sat, 13 Jun 2015 16:36:20 +0000 (17:36 +0100)
committerTim-Philipp Müller <tim@centricular.com>
Sat, 13 Jun 2015 16:59:42 +0000 (17:59 +0100)
This reverts commit 916b954315abc2f94348ec0be3e116c19b080b54.

Clearly something else was intended, and it also makes
more sense to add the extra bit. The resync marker is
N zero bits plus a 1 bit, and the pattern/mask needs to
be run on N+1 bits too.

(Even after the rever the code doesn't do that of course, so
it still needs to be fixed differently.)

https://bugzilla.gnome.org/show_bug.cgi?id=739345

gst-libs/gst/codecparsers/gstmpeg4parser.c

index 7dbc811..a2e63a2 100644 (file)
@@ -347,7 +347,7 @@ compute_resync_marker_size (const GstMpeg4VideoObjectPlane * vop,
     }
   }
 
-  return off;
+  return off++;                 /* Take the following 1 into account */
 }
 
 /**