[Bug 664274] [API] codecparsers: add gst_mpeg_video_parse_slice()

[GStreamer (bugzilla.gnome.org)]

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

--- Comment #2 from sreerenj <bsreerenj at gmail.com> 2013-01-25 13:25:29 UTC ---
(In reply to comment #1)
> Hrm, sorry for the long wait. So what's up with the
> 
>  /*Fixme: Not checking for 'Data Partitioning Mode' of Sequence Scalable
> Extension */
> 
> ? Isn't that API-relevant?
> 
> If I'm reading the spec right we'd need to pass an additional parameter with
> the sequence_scalable_extension (even if we don't have a struct for that yet or
> actually parse it).

I have a patch for Sequence Scalable Extension parsing. Do I need to file a
separate bug for that or is it enough to add the patch here.?

How about changing the API 
    gst_mpeg_video_parse_slice(Slice, 
                               Seq, SeqExt, SeqScalExt, 
                               data, size, offset); 

to

   gst_mpeg_video_parse_slice(Slice,
                              vertical_size, SeqScalExt,
                              data, size, offset);


Because now the mpegparser has gst_mpeg_video_finalise_mpeg2_sequence_header()
api which is calculating the vertical_size 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.