[Bug 732553] codecparsers: h264: fix identification of EOSEQ and EOS NALs

[GStreamer (bugzilla.gnome.org)]

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

Gwenole Beauchesne <gb.devel> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|                            |FIXED

--- Comment #6 from Gwenole Beauchesne <gb.devel at gmail.com> 2014-07-01 14:37:58 UTC ---
commit a61b7728b4d03f61c130d5b5df67e1903fe84a3b
Author: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
Date:   Tue Jul 1 16:09:28 2014 +0200

    codecparsers: h264: clarifications and documentation fixes.

    Fix documentation for GstH264NalUnit. The @ref_idc part was totally
    unbalanced. Also add a note about @offset and @size fields to remind
    that this is relative to the start of the NAL unit, thus including
    the header bytes.

commit 22b68b60ec7d83ef8487533cec3d4689b43bfd62
Author: Gwenole Beauchesne <gwenole.beauchesne at intel.com>
Date:   Tue Jul 1 15:46:17 2014 +0200

    codecparsers: h264: fix identification of EOSEQ and EOS NALs.

    An end_of_seq() [EOSEQ] or end_of_stream() [EOS] NAL unit is really
    one byte long because this shall include the NalHeaderBytes (1) too.
    The NALU.offset starts from the first byte of the header.

    This is the proper fix to commit d37f842. In practice, this fixes
    parsing of FRExt1_Panasonic_D and FRExt2_Panasonic_C, that include
    additional frames after an EOSEQ.

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

    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.