[Bug 526481] Support DVCPRO50 mov files

[GStreamer (bugzilla.gnome.org)]

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

Sebastian Dröge <slomo> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
 Attachment #110240|none                        |needs-work
             status|                            |

--- Comment #16 from Sebastian Dröge <slomo at circular-chaos.org> 2011-05-26 09:13:30 UTC ---
Review of attachment 110240:
 --> (https://bugzilla.gnome.org/review?bug=526481&attachment=110240)

::: ext/ffmpeg/gstffmpegcodecmap.c
@@ +472,3 @@
       } else {
         caps = gst_ff_vid_caps_new (context, codec_id, "video/x-dv",
+            "dvversion", GST_TYPE_INT_RANGE, 25, 50, NULL,

Shouldn't this get into two structures or a GST_TYPE_LIST instead? ffmpeg only
supports 25 or 50, right?

In that case just make it a GST_TYPE_LIST, see CODEC_ID_AAC in git for an
example how to do it

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