[vlc-devel] [PATCH] Add Dirac encoding support to VLC using the Schroedinger library.

[Anuradha Suraparaju]

Hi,

Thanks for the feedback. I'll send another patch soon. But I have a couple
of questions.


On Wed, Dec 8, 2010 at 4:48 AM, Jean-Baptiste Kempf <jb at videolan.org> wrote:

/**** snip ***/

>
>
> > +    set_capability( "encoder", 200 )
> Why 200 ?
>
> I just set it to the value used by the decoder. Is there any way pf
specifying that the Schro encoder module should be used ahead of the
Dirac-research encoder module to encode Dirac if both are present?

/*** snip ***/


>
>
> It seems that most of the code is options handling. Would it be possible
> to automatize this part and reduce the code size?
> I am not sure we need proper internalization of all the options, but
> maybe I am wrong.
>
>
Most of the options are advanced options that can be used to tweak the
Schroedinger encoder output. So the end user is unlikely to use them often.
I'll see if I can get rid of some of the advanced options. Schro encoder
does provide an interface to the encoding options. I'll see if I can use
that to reduce the code size.

I work part-time so it might be a couple of days before I send a new patch
in.


Regards,
Anuradha

Best Regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/
> +33 672 704 734
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20101208/d97c5a3b/attachment.html>