<div>Hello,</div><div><br></div>On Tue, Nov 2, 2010 at 12:14 AM, 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-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; ">
On Mon, Nov 01, 2010 at 07:22:45PM +0530, Srikanth Raju wrote :<br><div class="im">> > Why again a list of extensions?<br>> > Why in #define too?<br>> ><br>> This is to guess the type of a given media, without having the read the file<br>
> itself.<br></div>Sure, but still, my point was that we should have only one list of<br>extension. But let's ignore that for now and I'll fix it in a way or<br>another later.<br><div><div></div></div></blockquote>
<div><br></div>Yes for that we have to parse the other EXTENSIONS #define. Or fix code relating to the other #define to use the CSV define<div><br><div class="gmail_quote">2010/11/2 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Le lundi 1 novembre 2010 15:52:45 Srikanth Raju, vous avez écrit :<br>
<div class="im">> > * Using var_Create where not useful.<br>
><br>
> I don't understand why it's not useful...?<br>
<br>
</div>You don't seem to refer to the created variable ever other than to destroy it.<br>
Then why create it in the first place?<br>
<div class="im"><br></div></blockquote><div> </div><div>As I said, you will need to look at the other 3 patches for the complete module. I broke it up so it would be easier to read. But it seems to have caused some confusion. I should have been clearer.</div>
<div> </div></div><br>-- <br>Regards,<br>Srikanth Raju<br>
</div>