<div dir="ltr"><div><div><div><div>H Remi,<br><br></div>As discussed with Jean, multiple instances of same decoder can be created.<br></div>So in any case,we 'll need to serialize gst_init().<br></div>Will make it thread-safe.<br>
<br></div>Thanks,<br>Vikram<br></div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Apr 22, 2014 at 12:18 AM, ved kpl <span dir="ltr"><<a href="mailto:ved.kpl@gmail.com" target="_blank">ved.kpl@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div><div><div><div>Thanks Jean,<br><br></div>Remi, <br></div>As I mentioned earlier, gst_init() is not thread-safe, but calling it multiple times in a serialized manner is harmless. <br>
Since, currently we have only one gst based module, this problem should not concern us now.<br>
</div>But when we add more gst based modules, we'll need to take care of this.<br><br></div>Thanks,<br>Vikram<br><div><div><div><br></div></div></div></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra">
<br><br><div class="gmail_quote">On Mon, Apr 21, 2014 at 11:37 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 21 Apr, ved kpl wrote :<br>
<div>> > Vikram: It should be LGPL v 2.1, I guess here.  Need your inputs.<br>
<br>
</div>LGPL v2.1 +<br>
<div><br>
>    Vikram: Decodebin also adds gst video parsers before gst decoders which<br>
> can provide more information (GstCaps-wise) to the decoder. That was the<br>
> only reason I thought to<br>
>                have it. If you think it's not really doing any<br>
> value-addition, I'll remove it. Do let me know.<br>
<br>
</div>Either you explain clearly what it is or you remove it.<br>
<div><br>
> > Doesn't vlc_fourcc_GetCodecFromString do what you want already?<br>
> > Vikram: I see., Thanks! I will use it instead of this function. But one<br>
> > thing that comes to my mind now is YUYV format, gstreamer denotes it as<br>
> > YUY2.<br>
>                 Will need to handle such cases.<br>
<br>
</div>YUY2 is already in VLC as an alias to VLC_CODEC_YUYV. There should be no<br>
special cases.<br>
<br>
With my kindest regards,<br>
<div><div><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734" target="_blank">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>