<br><br><div class="gmail_quote">2010/8/1 Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span><br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Sat, Jul 31, 2010 at 09:17:31PM +0300, Kaarlo Räihä wrote :<br>
<div><div></div><div class="h5">> Some people have complained in forums that VLC tries to autoload .txt files<br>
> as subtitles since they have same kind of name as input file (<br>
> <a href="http://forum.videolan.org/viewtopic.php?f=14&t=55979#p244039" target="_blank">http://forum.videolan.org/viewtopic.php?f=14&t=55979#p244039</a> ). But because<br>
> those .txt files aren't subtitles, VLC will complain with dialog that says<br>
> 'format of 'something.txt' cannot be detected. Have a look at the log for<br>
> details'. So I made patch that removes .txt from supported subtitle file<br>
> extension list and users will avoid the dialogs.<br>
><br>
> I haven't heard that anyone uses .txt extension for subtitle file (at least<br>
> subbers avoid it) and thus this shouldn't cause issues for regular users<br>
> (QT4 subtitle loading dialog doesn't even offer .txt extension at all).<br>
> Problem is that same check function (subtitles_Filter) is used in control.c<br>
> (line 367) and with removal of .txt those subtitles cannot be loaded<br>
> anymore. There are better ways to solve the problem (like customizable<br>
> subtitle autoload extension list for users) but I would guess this easy<br>
> solution is good enough.<br>
<br>
</div></div>No, I don't think this is a good idea.<br>
Why not make the errors when loading of a subtitle silent when it is auto-detected?<br></blockquote><div><br></div><div>Because the dialog is created by InputSourceInit function in src/input/input.c which AFAIK has no information if the subtitle is autoloaded or not. And storing (+using) information like that for single error dialog in case like this is IMHO pointless and we would have to add subtitle specific code InputSourceInit which would make it look ugly. </div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
<br>
Best Regards,<br>
<font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a><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>
</font></blockquote></div><br>