Hi,<br><br>Thanks for the feedback. I'll send another patch soon. But I have a couple of questions.<br><br><br><div class="gmail_quote">On Wed, Dec 8, 2010 at 4:48 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br>
<br>/**** snip ***/<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"><br>
<br>
> +    set_capability( "encoder", 200 )<br>
Why 200 ?<br>
<br></blockquote><div>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?<br>
<br>/*** snip ***/<br> <br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<br>
<br>
It seems that most of the code is options handling. Would it be possible<br>
to automatize this part and reduce the code size?<br>
I am not sure we need proper internalization of all the options, but<br>
maybe I am wrong.<br>
<div><div></div><div class="h5"><br></div></div></blockquote><div><br>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.<br>
<br>I work part-time so it might be a couple of days before I send a new patch in.<br> <br><br>Regards,<br>Anuradha<br><br></div><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
<div><div class="h5">
Best Regards,<br>
<br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a><br>
+33 672 704 734<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</div></div></blockquote></div><br>