[Bug 693000] codecparsers: mpeg2: fix default quantizer matrix for intra blocks

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=693000
  GStreamer | gst-plugins-bad | git

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |RESOLVED
         Resolution|                            |FIXED
   Target Milestone|HEAD                        |1.0.6

--- Comment #22 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-02-06 17:06:28 UTC ---
Thanks!


commit 250555a7dec8f1a396a20994d28395249eb406d7
Author: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
Date:   Tue Feb 5 11:56:46 2013 +0100

    codecparsers: mpeg2: add helpers to convert quantization matrices.

    Add utility functions to convert quantization matrices from zigzag scan
    order (as encoded in the bitstream) into raster scan order. Also provide
    another function to reverse the operation.

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

    Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne at intel.com>

commit 46c01de05dbb50c74aca137298da5b3d16918034
Author: Cong Zhong <congx.zhong at intel.com>
Date:   Thu Jan 31 16:13:22 2013 +0800

    codecparsers: mpeg2: store quantization matrices in zigzag scan order.

    Quantizer matrices are encoded in zigzag scan order in the bitstream,
    but they are stored in raster scan order when they are parsed. However,
    default matrices were also prepared in zigzag scan order, hence the
    mismatch. i.e. the matrices were presented either in raster scan order
    if they are explicitly present in the bitstream, or they were presented
    in zigzag scan order if the default definitions were to be used instead.

    One way to solve this problem is to always expose the quantization
    matrices in zigzag scan order, since this is the role of the parser to
    not build up stories from the source bitstream and just present what
    is in there.

    Utility functions will be provided to convert quantization matrices in
    either scan order.

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

    Signed-off-by: Cong Zhong <congx.zhong at intel.com>
    Signed-off-by: Gwenole Beauchesne <gwenole.beauchesne at intel.com>

-- 
Configure bugmail: https://bugzilla.gnome.org/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug.
You are the assignee for the bug.