[Bug 694068] h264parser: Parse the cropping-rectangle separately.

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #12 from Gwenole Beauchesne <gb.devel at gmail.com> 2013-07-09 09:47:24 UTC ---
(In reply to comment #9)
> So this now still puts the display width/height (i.e. after cropping) in the
> caps via h264parse?

To recap,

This patche series (3 patches) provides:
- Un-cropped size in sps->width/height, as expected for buffer allocation ;
- Fix the calculation of the cropped width ;
- Make sure that h264parse provides cropped dimensions in size caps as it used
to do so before.

However, we clearly have an inconsistency between the mpegvideoparse and
h264parse elements. And this is not new. The mpegvideoparse element provides
uncropped dimensions in size caps, whereas the h264parse element provides
cropped dimensions in size caps. So, I suggest we open a new bug report for
this discrepancy, unless we can live with it.

In any case, the suggested patch series here now maintains the original
h264parse behaviour, while extending the codec parser to provide the correct
cropped dimensions in separate fields, and the original (uncropped) dimensions
in plain width/height fields.

Thanks.

-- 
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.