[Bug 748507] New: mpegtsmux: setting pes payload length to pes header in case of video ES

[GStreamer (GNOME Bugzilla)]

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

            Bug ID: 748507
           Summary: mpegtsmux: setting pes payload length to pes header in
                    case of video ES
    Classification: Platform
           Product: GStreamer
           Version: git master
                OS: All
            Status: NEW
          Severity: minor
          Priority: Normal
         Component: gst-plugins-bad
          Assignee: gstreamer-bugs at lists.freedesktop.org
          Reporter: zzoon.ko at samsung.com
        QA Contact: gstreamer-bugs at lists.freedesktop.org
     GNOME version: ---

Currently, in case of video ES, it always set 0 to the length of pes.
According to ISO spec, this is right, and working fine.
But doesn't it need to set length when the length is lesser than 16bit max (in
case of video ES)?

Is there any specific reason to set 0 in all video ES?
How about this logic?

if [ length is bigger than 16bit max ]
  set to 0
else 
  set to length

I'm using mpegtsmux in my project, and I'm thinking about this change.
Could you give me some advices for this?

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are the assignee for the bug.