[Bug 679878] [0.11][videodecoder] Need explicit negotiation method

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=679878
  GStreamer | gst-plugins-base | 0.11.x

Edward Hervey <bilboed> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |bilboed at gmail.com
          Component|gst-plugins-ugly            |gst-plugins-base
            Summary|[0.11] mpeg2dec: cropping   |[0.11][videodecoder] Need
                   |broken:                     |explicit negotiation method
                   |gst_video_decoder_alloc_out |
                   |put_frame: assertion        |
                   |`frame->output_buffer ==    |
                   |NULL' failed                |

--- Comment #1 from Edward Hervey <bilboed at gmail.com> 2012-07-16 08:57:44 UTC ---
The problem is due to the way negotiation works with the base classes.

When you set the output format (which may include striding and whatsoever), the
call to _set_caps and _negotiate doesn't happen before you call buffer_alloc or
push()...

It looks like we might need to reinstate an explicit
gst_video_decoder_negotiate() method for that.

It's also very likely this is the cause PAR isn't correct with some video feeds
also.

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