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

Anuradha Suraparaju asuraparaju at gmail.com
Wed Dec 8 04:43:21 CET 2010


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


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>

More information about the vlc-devel mailing list