[vlc-devel] [PATCH 1/1] codec: Add GStreamer based video decoder module
ved.kpl at gmail.com
Tue Apr 22 07:49:52 CEST 2014
As discussed with Jean, multiple instances of same decoder can be created.
So in any case,we 'll need to serialize gst_init().
Will make it thread-safe.
On Tue, Apr 22, 2014 at 12:18 AM, ved kpl <ved.kpl at gmail.com> wrote:
> Thanks Jean,
> 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.
> 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
>> > 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
>> > > thing that comes to my mind now is YUYV format, gstreamer denotes it
>> > > 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:
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the vlc-devel