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

[ved kpl]

H Remi,

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.

Thanks,
Vikram


On Tue, Apr 22, 2014 at 12:18 AM, ved kpl <ved.kpl at gmail.com> wrote:

> 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/4a2c0876/attachment.html>