mpegvideoparse:Correct the skipping of vbv_delay in picture header and some typo...
authorSreerenj Balachandran <sreerenj.balachandran@intel.com>
Thu, 3 Nov 2011 12:09:52 +0000 (14:09 +0200)
committerThibault Saunier <thibault.saunier@collabora.com>
Mon, 14 Nov 2011 11:32:32 +0000 (08:32 -0300)
https://bugzilla.gnome.org/show_bug.cgi?id=663309

gst-libs/gst/codecparsers/gstmpegvideoparser.c
gst-libs/gst/codecparsers/gstmpegvideoparser.h

index 2d762fb..85d8b1d 100644 (file)
@@ -681,8 +681,8 @@ gst_mpeg_video_parse_picture_header (GstMpegVideoPictureHdr * hdr,
   if (hdr->pic_type == 0 || hdr->pic_type > 4)
     goto failed;                /* Corrupted picture packet */
 
-  /* skype VBV delay */
-  if (!gst_bit_reader_skip (&br, 8))
+  /* skip VBV delay */
+  if (!gst_bit_reader_skip (&br, 16))
     goto failed;
 
   if (hdr->pic_type == GST_MPEG_VIDEO_PICTURE_TYPE_P
index 444092e..7c23114 100644 (file)
@@ -38,7 +38,7 @@ G_BEGIN_DECLS
 /**
  * GstMpegVideoPacketTypeCode:
  * @GST_MPEG_VIDEO_PACKET_PICTURE: Picture packet starting code
- * @GST_MPEG_VIDEO_PACKET_SLICE_MIN: Picture packet starting code
+ * @GST_MPEG_VIDEO_PACKET_SLICE_MIN: Slice min packet starting code
  * @GST_MPEG_VIDEO_PACKET_SLICE_MAX: Slice max packet starting code
  * @GST_MPEG_VIDEO_PACKET_USER_DATA: User data packet starting code
  * @GST_MPEG_VIDEO_PACKET_SEQUENCE : Sequence packet starting code
@@ -186,7 +186,7 @@ typedef struct _GstMpegVideoTypeOffsetSize  GstMpegVideoTypeOffsetSize;
  * @width: Width of each frame
  * @height: Height of each frame
  * @par_w: Calculated Pixel Aspect Ratio width
- * @par_h: Pixel Aspect Ratio height
+ * @par_h: Calculated Pixel Aspect Ratio height
  * @fps_n: Calculated Framrate nominator
  * @fps_d: Calculated Framerate denominator
  * @bitrate_value: Value of the bitrate as is in the stream (400bps unit)