Hello,<div>I've attached an improved patch.<br><br><div class="gmail_quote">On Mon, Oct 18, 2010 at 12:18 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">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 Mon, Oct 18, 2010 at 01:09:21AM +0530, Srikanth Raju wrote :<br>
> +#define EXTENSIONS_AUDIO_CSV "a52", "aac", "ac3", "ape", "dts", "flac", "it", \<br>
> +                         "m4a", "m4p", "mka", "mlp", "mod", "mp1", "mp2", "mp3",\<br>
> +                         "oga", "ogg", "oma", "s3m", "spx" \<br>
> +                         "wav", "wma", "wv", "xm"<br>
> +<br>
> +#define EXTENSIONS_VIDEO_CSV "asf", "avi", "divx", "dv", "flv", "gxf", "iso", \<br>
> +                             "m1v", "m2v", "m2t", "m2ts", "m4v", "mkv", "mov",\<br>
> +                             "mp2", "mp4", "mpeg", "mpeg1", \<br>
> +                             "mpeg2", "mpeg4", "mpg", "mts", "mxf", "nuv", \<br>
> +                             "ogg", "ogm", "ogv", "ogx", "ps", \<br>
> +                             "rec", "rm", "rmvb", "ts", "vob", "wmv"<br>
<br>
Why again a list of extensions?<br>
Why in #define too?<br><br></blockquote><div> </div><div>This is to guess the type of a given media, without having the read the file itself.</div><div><br></div><div><meta http-equiv="content-type" content="text/html; charset=utf-8"><meta http-equiv="content-type" content="text/html; charset=utf-8"><br class="Apple-interchange-newline">
2010/10/19 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
* Using var_Create where not useful. </blockquote><div>I don't understand why it's not useful...? </div><div><br></div><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
* I can't even find the thread function. Does this code links at all??<br></blockquote><div>No! Please look at the other 3 patches for the rest of the functions! This module isn't enabled in <a href="http://configure.ac">configure.ac</a>, so it won't break anything.</div>
<div><br></div><blockquote class="gmail_quote" style="margin-top: 0px; margin-right: 0px; margin-bottom: 0px; margin-left: 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; ">
* The big IF_CMP/ELSE_CMP list should be replaced with a static mapping table<br>and a binary search. It would also remove the need for the huge switch.<br></blockquote></div></div>I don't see how the switch can be removed without having lots of setter functions.<div>
<br>-- <br>Regards,<br>Srikanth Raju<br>
</div></div>