GNOME Bugzilla – Bug 733872
codecparsers: mpegvideo: Add more fields to the headers.
Last modified: 2016-04-14 15:51:28 UTC
Adding a couple of missing fields to SequenceHdr, PictureHdr and SliceHdr
Created attachment 281872 [details] [review] codecparsers: mpegvideo: Add more fields to the GstMpegVideoSequenceHdr
Created attachment 281873 [details] [review] codecparsers: mpegvideo: Add vbv_delay field to PictureHeader
Created attachment 281874 [details] [review] codecparsers: mpegvideo: Parse more slice header fields
Thanks, pushed to master (sorry for the absurd delay): commit 8f853cfed59fb0a38c4e5b534ab7fbe22682ca9c Author: Sreerenj Balachandran <sreerenj.balachandran@intel.com> Date: Mon Jul 28 12:14:02 2014 +0300 codecparsers: mpegvideo: Parse more slice header fields https://bugzilla.gnome.org/show_bug.cgi?id=733872 commit 51c75260305f171ea5b760f159451296df35895f Author: Sreerenj Balachandran <sreerenj.balachandran@intel.com> Date: Mon Jul 28 11:17:05 2014 +0300 codecparsers: mpegvideo: Add vbv_delay field to PictureHeader https://bugzilla.gnome.org/show_bug.cgi?id=733872 commit 7b0b74041ed24b4b7ab4cfb0217059cd88f4b72e Author: Sreerenj Balachandran <sreerenj.balachandran@intel.com> Date: Mon Jul 28 11:14:22 2014 +0300 codecparsers: mpegvideo: Add more fields to the GstMpegVideoSequenceHdr Added the fields load_non_intra_quantiser_matrix and load_chroma_intra_quantiser_matrix to the sequence header structure. https://bugzilla.gnome.org/show_bug.cgi?id=733872
This breaks ABI of course, just for the record. Which is ok, but should make sure we add a note to the 1.10 release notes about that ;)
Aha very old patches, was not even in my distant memory ! Added those for codec analyzer in gst-devtools.