]> git.sesse.net Git - ffmpeg/commit
avformat/matroskaenc: Don't implicitly mark WebVTT in WebM as English
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Sat, 18 Jan 2020 01:19:46 +0000 (02:19 +0100)
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>
Thu, 2 Apr 2020 18:40:44 +0000 (20:40 +0200)
commitda4ba2431c92017379a0bd0b3c0fb74ce6093f09
tree523c8f1c776ac4753034aff8f66ad6844343861e
parentbc52ce309dfcec6151bf1776e6a256f547285700
avformat/matroskaenc: Don't implicitly mark WebVTT in WebM as English

Writing the language of WebVTT in WebM proceeded differently than the
language of all other tracks: In case no language was given, it does not
write anything instead of "und" (for undefined). Because the default
value of the Language element in WebM (that inherited it from Matroska)
is "eng" (for English), any such track will actually be flagged as
English.

Doing it this way goes back to commit 509642b4 (the commit adding
support for WebVTT) and no reason for this has been given in the commit
message or in the discussion about this patch on the mailing list; the
best I can think of is this: the WebM wiki contains "The srclang attribute
is stored as the Language sub-element." Someone unfamiliar with default
values in Matroska/WebM could interpret this as meaning that no Language
element should be written if the language is unknown. And this is wrong
and this commit changes it.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
libavformat/matroskaenc.c