[FFmpeg-devel] [PATCH]Change isom language code for dutch to nld

Michael Niedermayer michaelni at gmx.at
Tue Mar 18 19:39:30 CET 2014


On Tue, Mar 18, 2014 at 01:42:07PM +0100, Carl Eugen Hoyos wrote:
> Hi!
> 
> Attached patch intends to fix ticket #3375.
> I was able to verify that "nld" is the correct language code following 
> ISO-639-2/T:
> http://www-01.sil.org/iso639-3/codes.asp?order=639_3&letter=n
> I have no idea if mp4 (and mov) prefer ISO-639-2/T over ISO-639-2/B.
> 
> Please review, Carl Eugen

>  isom.c |    2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 1035d98c53867b9bf0b4ce82a0ab68c0176f99b0  patchnld.diff
> diff --git a/libavformat/isom.c b/libavformat/isom.c
> index aec6878..ba63a37 100644
> --- a/libavformat/isom.c
> +++ b/libavformat/isom.c
> @@ -323,7 +323,7 @@ const AVCodecTag ff_codec_movsubtitle_tags[] = {
>  /* deprecated by putting the code as 3*5bit ascii */
>  static const char mov_mdhd_language_map[][4] = {
>      /* 0-9 */
> -    "eng", "fra", "ger", "ita", "dut", "sve", "spa", "dan", "por", "nor",
> +    "eng", "fra", "ger", "ita", "nld", "sve", "spa", "dan", "por", "nor",

"ger" and "nld" are not of the same kind AFAIK so this looks wrong

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140318/a7c2bc02/attachment.asc>


More information about the ffmpeg-devel mailing list