summaryrefslogtreecommitdiffstats
path: root/player/external_files.c
diff options
context:
space:
mode:
authorwm4 <wm4@nowhere>2017-02-03 11:32:16 +0100
committerwm4 <wm4@nowhere>2017-02-03 11:32:16 +0100
commit50991fac81eb56bc0985af2c9d31cd503302dcfe (patch)
treeff6cd194905b936643b1fd6aaff0382b26af7101 /player/external_files.c
parent1ec6cd89b96a8eca5f89316373a5d12f4cb828f5 (diff)
downloadmpv-50991fac81eb56bc0985af2c9d31cd503302dcfe.tar.bz2
mpv-50991fac81eb56bc0985af2c9d31cd503302dcfe.tar.xz
sub: remove .txt as text subtitle extension
If used with fuzzy matching, the player tends to pick up random text files, sometimes with interesting results. The most interesting interaction is when the user uses --log-file=something.txt, and mpv tries to open its own log file. It essentially "freezes" during probing, because every time it reads from it, it will write some more data, which in turn will cause more data to be read - until the 2MB max. probing size is slowly reached. This is not even an obscure corner case, but happened to multiple users. The .txt extension has been considered a subtitle extension ever since the code was added to MPlayer's subreader.c, but I'm not seeing many actual subtitle files with this extension, so just get rid of it.
Diffstat (limited to 'player/external_files.c')
-rw-r--r--player/external_files.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/player/external_files.c b/player/external_files.c
index fa0d6c0336..5eb3a1d730 100644
--- a/player/external_files.c
+++ b/player/external_files.c
@@ -16,7 +16,7 @@
#include "external_files.h"
static const char *const sub_exts[] = {"utf", "utf8", "utf-8", "idx", "sub", "srt",
- "smi", "rt", "txt", "ssa", "aqt", "jss",
+ "smi", "rt", "ssa", "aqt", "jss",
"js", "ass", "mks", "vtt", "sup", "scc",
NULL};