[gstreamer-bugs] [Bug 613789] New: [jpegenc] Updating caps is not supported

[GStreamer (bugzilla.gnome.org)]

https://bugzilla.gnome.org/show_bug.cgi?id=613789
  GStreamer | gst-plugins-good | 0.10.21

           Summary: [jpegenc] Updating caps is not supported
    Classification: Desktop
           Product: GStreamer
           Version: 0.10.21
        OS/Version: All
            Status: UNCONFIRMED
          Severity: normal
          Priority: Normal
         Component: gst-plugins-good
        AssignedTo: gstreamer-bugs at lists.sourceforge.net
        ReportedBy: raimo.jarvi at gmail.com
         QAContact: gstreamer-bugs at lists.sourceforge.net
      GNOME target: ---
     GNOME version: ---


Created an attachment (id=156964)
 View: https://bugzilla.gnome.org/attachment.cgi?id=156964
 Review: https://bugzilla.gnome.org/review?bug=613789&attachment=156964

Fix updating jpegenc caps

The jpegenc element apparently doesn't support updating caps of the sink pad. I
have a pipeline where jpegenc is used for taking snapshots of the video and
when video caps change (e.g. video aspect ratio changes), there is a
"not-negotiated" error (apparently because jpegenc rejects the new caps).

Attached is a proposed patch that fixes this. I basically copied the behaviour
from pngenc, I don't know if it's the right approach. The patch does the
following:

- Remove getcaps function
- Don't use fixed caps for sink pad
- Simplify setcaps function, because it's only used for sink pad

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