[Bug 692461] codecparsers: vc1: fix bitplanes decoding (DIFF6 or NORM6)

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=692461
  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.1.1

--- Comment #6 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2013-01-25 12:34:44 UTC ---
commit 10639eb88972bf89656af694b8bf8c4b41255c69
Author: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
Date:   Thu Jan 24 18:14:28 2013 +0100

    codecparsers: vc1: fix bitplanes decoding (DIFF6 or NORM6 residual bytes).

    Fix parsing of residual bytes. This is a two-step process. First,
    remaining colums of full vertical resolution (<height>) need to be
    processed. Next, remaining bytes in the first row can be processed,
    while taking into account the fact that we may have filled in the
    first columns already. So, this is not full horizontal resolution.

    The following figure helps in understanding the expected order of
    operations, for a 8x5 MBs bitplane.

        5 5 6 6 6 6 6 6
        5 5 1 1 1 2 2 2
        5 5 1 1 1 2 2 2
        5 5 3 3 3 4 4 4
        5 5 3 3 3 4 4 4

    So, after tiles 1 to 4 are decoded, vertical tile 5 needs to be
    processed (2x5 MBs) and then the horizontal tile 6 (6x1 MBs).

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

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

commit fa2a526f04b1f51201fbb837fa430f93609938e8
Author: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
Date:   Thu Jan 24 17:28:22 2013 +0100

    codecparsers: vc1: fix bitplanes decoding (DIFF6 or NORM6).

    Fix decoding of DIFF6 or NORM6 bitplanes with an odd number of lines
    (3x2 "horizontal" tiles). In this case, we have to skip the first line
    of macroblocks but <width> number of bytes was used to do so, instead
    of the actual <stride> size.

    This fixes decoding for the video sample attached to:
    https://bugzilla.gnome.org/show_bug.cgi?id=668565

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

    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.