[Bug 673925] [API] codecparsers: add JPEG baseline parser

[GStreamer (bugzilla.gnome.org)]

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

Tim-Philipp Müller <t.i.m> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
                 CC|                            |t.i.m at zen.co.uk
            Summary|codecparsers: add JPEG      |[API] codecparsers: add
                   |baseline parser             |JPEG baseline parser
     Ever Confirmed|0                           |1

--- Comment #3 from Tim-Philipp Müller <t.i.m at zen.co.uk> 2012-07-10 22:23:05 UTC ---
> I have also tried to update the API so that to align with existing codecparsers
> style like MPEG-2 video parser in particular. Here is another gstjpegparser.h
> proposal. I have not tried to migrate the existing gstjpegparser.c code to it
> yet. Please tell us what you prefer. I think we could also find an intermediate
> version that would simplify porting.

Unfortunately the mpeg-2 video parser was not the best example of the kind of
API we prefer.

Could you have a look at how the API has been refactored recently and update
the jpeg parser?

Mostly:
  - no GList * return for _parse()
  - plain gboolean return values 

Not sure the GstJpegProfile enum is entirely right, it's a bit unusual to
combine enums and flags in the same enumeration, that's going to trip up
bindings too. And I am not sure 'profile' is the right term here. But this is
already well into bikeshedding territory, the other issues are more important.

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