<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">
<html xmlns="http://www.w3.org/1999/xhtml">
<head>
  <meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
  <meta http-equiv="Content-Style-Type" content="text/css" />
  <meta name="generator" content="pandoc" />
  <title></title>
  <style type="text/css">code{white-space: pre;}</style>
</head>
<body>
<p>Hi j-b,</p>
<p>On 2017-05-19 15:06, Jean-Baptiste Kempf wrote:</p>
<blockquote style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;color:#500050">
<pre><code> XML is way too vague
 ---
  modules/demux/directory.c | 2 +-
  1 file changed, 1 insertion(+), 1 deletion(-)

 diff --git a/modules/demux/directory.c b/modules/demux/directory.c
 index ffc87b0fd6..88d42fdf91 100644
 --- a/modules/demux/directory.c
 +++ b/modules/demux/directory.c
 @@ -121,7 +121,7 @@ vlc_module_begin()
          change_string_list( psz_recursive_list, psz_recursive_list_text )
      add_string( "ignore-filetypes", "m3u,db,nfo,ini,jpg,jpeg,ljpg,gif,png,pgm,"
                  "pgmyuv,pbm,pam,tga,bmp,pnm,xpm,xcf,pcx,tif,tiff,lbm,sfv,txt,"
 -                "sub,idx,srt,cue,ssa",
 +                "sub,idx,srt,cue,ssa,xml",
                  IGNORE_TEXT, IGNORE_LONGTEXT, false )
      add_bool( "show-hiddenfiles", false,
                SHOW_HIDDENFILES_TEXT, SHOW_HIDDENFILES_LONGTEXT, false )
 -- </code></pre>
</blockquote>
<p>As written in <code>#videolan</code> at <code>freenode</code>, this would break playback for <a href="https://support.apple.com/en-us/HT201610">iTunes Library Files</a> (<a href="http://git.videolan.org/?p=vlc.git;a=blob;f=modules/demux/playlist/itml.c;h=5f636262f1b9248eaedb3dfce354592c946884fe;hb=HEAD">demux/playlist/itml.c</a>) when opening a directory (as they would no longer be added to the playlist).</p>
<p>One could, potentially, ignore files (remove from playlist) for which <em>preparsing</em> (if any) fails for a certain set of file-extensions, but it will probably lead to users being confused about what is happening to the state of the playlist (as they will most likely be able to see the items disappear in real time).</p>
<p>Having it as an opt-in option might be a solution, but I am not too sure. Personally, I am semi-against making decisions based on the file-extension alone (as it does not necessarily reflect the true contents of the file). I do however see how users can find the <code>--hidden-filetypes</code> option useful.</p>
<p>Best Regards,<br />
Filip</p>
</body>
</html>