[vlc-devel] [PATCH 1/1] codec: Add GStreamer based video decoder module

[ved kpl]

Thanks Jean,

Remi,
As I mentioned earlier, gst_init() is not thread-safe, but calling it
multiple times in a serialized manner is harmless.
Since, currently we have only one gst based module, this problem should not
concern us now.
But when we add more gst based modules, we'll need to take care of this.

Thanks,
Vikram



On Mon, Apr 21, 2014 at 11:37 PM, Jean-Baptiste Kempf <jb at videolan.org>wrote:

> On 21 Apr, ved kpl wrote :
> > > Vikram: It should be LGPL v 2.1, I guess here.  Need your inputs.
>
> LGPL v2.1 +
>
> >    Vikram: Decodebin also adds gst video parsers before gst decoders
> which
> > can provide more information (GstCaps-wise) to the decoder. That was the
> > only reason I thought to
> >                have it. If you think it's not really doing any
> > value-addition, I'll remove it. Do let me know.
>
> Either you explain clearly what it is or you remove it.
>
> > > Doesn't vlc_fourcc_GetCodecFromString do what you want already?
> > > Vikram: I see., Thanks! I will use it instead of this function. But one
> > > thing that comes to my mind now is YUYV format, gstreamer denotes it as
> > > YUY2.
> >                 Will need to handle such cases.
>
> YUY2 is already in VLC as an alias to VLC_CODEC_YUYV. There should be no
> special cases.
>
> With my kindest regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20140422/afb8bfaf/attachment.html>