From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.mplayerhq.hu (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id E974F4D233 for ; Tue, 18 Feb 2025 16:56:54 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C014668BFB2; Tue, 18 Feb 2025 18:56:50 +0200 (EET) Received: from glom.nmugroup.com (glom.nmugroup.com [193.183.80.6]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 3E7EF68BE4C for ; Tue, 18 Feb 2025 18:56:43 +0200 (EET) Received: from localhost (localhost [127.0.0.1]) by glom.nmugroup.com (Postfix) with ESMTP id 44F95542AA15 for ; Tue, 18 Feb 2025 17:56:42 +0100 (CET) Received: from debian.lan (unknown [IPv6:2a00:66c0:a::72c]) (Authenticated sender: git01) by glom.nmugroup.com (Postfix) with ESMTPSA id F21BB5421F63 for ; Tue, 18 Feb 2025 17:56:41 +0100 (CET) Message-ID: From: Tomas =?ISO-8859-1?Q?H=E4rdin?= To: FFmpeg development discussions and patches Date: Tue, 18 Feb 2025 17:56:40 +0100 In-Reply-To: References: Content-Type: multipart/mixed; boundary="=-ZIMt7PTEoCcrNNeMnaCw" User-Agent: Evolution 3.46.4-2 MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [RFC] Documenting metadata keys, informative (non-copied) metadata X-BeenThere: ffmpeg-devel@ffmpeg.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: FFmpeg development discussions and patches List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Reply-To: FFmpeg development discussions and patches Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: --=-ZIMt7PTEoCcrNNeMnaCw Content-Type: text/plain; charset="UTF-8" Content-Transfer-Encoding: quoted-printable tor 2025-02-13 klockan 12:54 +0100 skrev Tomas H=C3=A4rdin: > Hi >=20 > In the samples_md5 patch discussion Michael wanted the proposed key > to > be documented. But it turns out we don't have any documentation for > metadata keys! So I'm starting this thread to talk about it. I reckon > we create a new file called doc/metadata_keys.texi with a table > listing > keys and where they can appear (format, stream, frames), sorted by > name, with a brief description of each one. Any keys that require > more > detailed documentation we could add sections for. >=20 > Another issue raised was that some metadata keys shouldn't be carried > over automatically to muxers. In the samples_md5 thread it was > pointed > out by Andreas that we don't want to mux that in AIFF. It was also > pointed out that it stops being valid if the audio is cut. This isn't > the first time I've come across cases where we don't want metadata to > be copied, so I'm taking the opportunity to propose informative > output- > only metadata could reside in their own namespace. I propose info: > for > that, so info:samples_md5 in this specific case, or maybe just > info:md5. HEVC frames could similarly have such metadata applied. Attached is a PoC for the latter part. Unlike the previous samples_md5 patch, this one only requires updating a FATE ref concerning a .flac file. I have to go, else I'd also post the documentation thing as well. /Tomas --=-ZIMt7PTEoCcrNNeMnaCw Content-Disposition: attachment; filename="0001-lavu-dict-Add-info-namespace-and-AV_DICT_COPY_INFO.patch" Content-Type: text/x-patch; name="0001-lavu-dict-Add-info-namespace-and-AV_DICT_COPY_INFO.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 RnJvbSAyOTMwZDQyYzFkYmExZjYwOGMxMDU4Yjk1ZWJjZmYzOTA5Njk1NzE1IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiA9P1VURi04P3E/VG9tYXM9MjBIPUMzPUE0cmRpbj89IDxnaXRA aGFlcmRpbi5zZT4KRGF0ZTogVHVlLCAxOCBGZWIgMjAyNSAxNjo0NTo0MSArMDEwMApTdWJqZWN0 OiBbUEFUQ0ggMS8zXSBsYXZ1L2RpY3Q6IEFkZCBpbmZvOiBuYW1lc3BhY2UgYW5kIEFWX0RJQ1Rf Q09QWV9JTkZPCgpUaGUgaW50ZW50IGlzIHRvIGVuYWJsZSBkZW11eGVycyB0byBwcm92aWRlIGlu Zm9ybWF0aXZlIG1ldGFkYXRhIHRoYXQgaXMgbm90IHRvIGJlIGNvcGllZCB0byBtdXhlcnMKLS0t CiBsaWJhdnV0aWwvZGljdC5jIHwgMyArKysKIGxpYmF2dXRpbC9kaWN0LmggfCAxICsKIDIgZmls ZXMgY2hhbmdlZCwgNCBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEvbGliYXZ1dGlsL2RpY3Qu YyBiL2xpYmF2dXRpbC9kaWN0LmMKaW5kZXggNmZiMDkzOTliYS4uMDY3MTA3YjdjZCAxMDA2NDQK LS0tIGEvbGliYXZ1dGlsL2RpY3QuYworKysgYi9saWJhdnV0aWwvZGljdC5jCkBAIC0yMzksOSAr MjM5LDEyIEBAIGludCBhdl9kaWN0X2NvcHkoQVZEaWN0aW9uYXJ5ICoqZHN0LCBjb25zdCBBVkRp Y3Rpb25hcnkgKnNyYywgaW50IGZsYWdzKQogICAgIGNvbnN0IEFWRGljdGlvbmFyeUVudHJ5ICp0 ID0gTlVMTDsKIAogICAgIHdoaWxlICgodCA9IGF2X2RpY3RfaXRlcmF0ZShzcmMsIHQpKSkgewor ICAgICAgICAvLyBkbyBub3QgY29weSBpbmZvOiBrZXlzIHVubGVzcyBleHBsaWNpdGx5IHRvbGQg dG8gZG8gc28KKyAgICAgICAgaWYgKChmbGFncyAmIEFWX0RJQ1RfQ09QWV9JTkZPKSB8fCBzdHJu Y21wKHQtPmtleSwgImluZm86IiwgNSkpIHsKICAgICAgICAgaW50IHJldCA9IGF2X2RpY3Rfc2V0 KGRzdCwgdC0+a2V5LCB0LT52YWx1ZSwgZmxhZ3MpOwogICAgICAgICBpZiAocmV0IDwgMCkKICAg ICAgICAgICAgIHJldHVybiByZXQ7CisgICAgICAgIH0KICAgICB9CiAKICAgICByZXR1cm4gMDsK ZGlmZiAtLWdpdCBhL2xpYmF2dXRpbC9kaWN0LmggYi9saWJhdnV0aWwvZGljdC5oCmluZGV4IDcx M2M5ZTM2MWEuLjUwMTIzNzRkMjYgMTAwNjQ0Ci0tLSBhL2xpYmF2dXRpbC9kaWN0LmgKKysrIGIv bGliYXZ1dGlsL2RpY3QuaApAQCAtODIsNiArODIsNyBAQAogI2RlZmluZSBBVl9ESUNUX0FQUEVO RCAgICAgICAgIDMyICAgLyoqPCBJZiB0aGUgZW50cnkgYWxyZWFkeSBleGlzdHMsIGFwcGVuZCB0 byBpdC4gIE5vdGUgdGhhdCBubwogICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAgICAg ICAgICBkZWxpbWl0ZXIgaXMgYWRkZWQsIHRoZSBzdHJpbmdzIGFyZSBzaW1wbHkgY29uY2F0ZW5h dGVkLiAqLwogI2RlZmluZSBBVl9ESUNUX01VTFRJS0VZICAgICAgIDY0ICAgLyoqPCBBbGxvdyB0 byBzdG9yZSBzZXZlcmFsIGVxdWFsIGtleXMgaW4gdGhlIGRpY3Rpb25hcnkgKi8KKyNkZWZpbmUg QVZfRElDVF9DT1BZX0lORk8gICAgIDEyOCAgIC8qKjwgSW4gYXZfZGljdF9jb3B5KCksIGFsc28g Y29weSBpbmZvOiBrZXlzICovCiAvKioKICAqIEB9CiAgKi8KLS0gCjIuMzkuNQoK --=-ZIMt7PTEoCcrNNeMnaCw Content-Disposition: attachment; filename="0002-lavu-dict-Indent.patch" Content-Type: text/x-patch; name="0002-lavu-dict-Indent.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 RnJvbSBhMzgxZTllMjY5NTVjZDZiMmQxMDVjZjBmM2JjYjNiNjE5OTdmZWJlIE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiA9P1VURi04P3E/VG9tYXM9MjBIPUMzPUE0cmRpbj89IDxnaXRA aGFlcmRpbi5zZT4KRGF0ZTogVHVlLCAxOCBGZWIgMjAyNSAxNjo0NTo1OSArMDEwMApTdWJqZWN0 OiBbUEFUQ0ggMi8zXSBsYXZ1L2RpY3Q6IEluZGVudAoKLS0tCiBsaWJhdnV0aWwvZGljdC5jIHwg NiArKystLS0KIDEgZmlsZSBjaGFuZ2VkLCAzIGluc2VydGlvbnMoKyksIDMgZGVsZXRpb25zKC0p CgpkaWZmIC0tZ2l0IGEvbGliYXZ1dGlsL2RpY3QuYyBiL2xpYmF2dXRpbC9kaWN0LmMKaW5kZXgg MDY3MTA3YjdjZC4uZTUzYWI0ZTU1NCAxMDA2NDQKLS0tIGEvbGliYXZ1dGlsL2RpY3QuYworKysg Yi9saWJhdnV0aWwvZGljdC5jCkBAIC0yNDEsOSArMjQxLDkgQEAgaW50IGF2X2RpY3RfY29weShB VkRpY3Rpb25hcnkgKipkc3QsIGNvbnN0IEFWRGljdGlvbmFyeSAqc3JjLCBpbnQgZmxhZ3MpCiAg ICAgd2hpbGUgKCh0ID0gYXZfZGljdF9pdGVyYXRlKHNyYywgdCkpKSB7CiAgICAgICAgIC8vIGRv IG5vdCBjb3B5IGluZm86IGtleXMgdW5sZXNzIGV4cGxpY2l0bHkgdG9sZCB0byBkbyBzbwogICAg ICAgICBpZiAoKGZsYWdzICYgQVZfRElDVF9DT1BZX0lORk8pIHx8IHN0cm5jbXAodC0+a2V5LCAi aW5mbzoiLCA1KSkgewotICAgICAgICBpbnQgcmV0ID0gYXZfZGljdF9zZXQoZHN0LCB0LT5rZXks IHQtPnZhbHVlLCBmbGFncyk7Ci0gICAgICAgIGlmIChyZXQgPCAwKQotICAgICAgICAgICAgcmV0 dXJuIHJldDsKKyAgICAgICAgICAgIGludCByZXQgPSBhdl9kaWN0X3NldChkc3QsIHQtPmtleSwg dC0+dmFsdWUsIGZsYWdzKTsKKyAgICAgICAgICAgIGlmIChyZXQgPCAwKQorICAgICAgICAgICAg ICAgIHJldHVybiByZXQ7CiAgICAgICAgIH0KICAgICB9CiAKLS0gCjIuMzkuNQoK --=-ZIMt7PTEoCcrNNeMnaCw Content-Disposition: attachment; filename="0003-libavformat-flacdec-Export-samples-md5-as-metadata.patch" Content-Type: text/x-patch; name="0003-libavformat-flacdec-Export-samples-md5-as-metadata.patch"; charset="UTF-8" Content-Transfer-Encoding: base64 RnJvbSBlZjUzNDkzZTA1OWU4Zjk1MmNkYjlmYWEzMzhkMWM3MGI4Y2I5ODU2IE1vbiBTZXAgMTcg MDA6MDA6MDAgMjAwMQpGcm9tOiBNYXR0aWFzIFdhZG1hbiA8d2FkZXJAc3BvdGlmeS5jb20+CkRh dGU6IE1vbiwgMTEgT2N0IDIwMjEgMTU6Mzg6MTMgKzAyMDAKU3ViamVjdDogW1BBVENIIDMvM10g bGliYXZmb3JtYXQvZmxhY2RlYzogRXhwb3J0IHNhbXBsZXMgbWQ1IGFzIG1ldGFkYXRhCgpXaWxs IGJlIHVzZWQgYnkgbWFsIHRvIGNvbXBhcmUgbWV0YWRhdCBtZDUgd2l0aCBkZWNvZGVkIHNhbXBs ZXMgbWQ1LgoKUGFydCBvZiBmaXhpbmcgaHR0cHM6Ly9qaXJhLnNwb3RpZnkubmV0L2Jyb3dzZS9H T0wtNjgxCi0tLQogbGliYXZmb3JtYXQvZmxhY2RlYy5jICAgICAgICAgICAgICAgfCA2ICsrKysr KwogdGVzdHMvcmVmL2ZhdGUvY292ZXItYXJ0LWZsYWMtcmVtdXggfCAxICsKIDIgZmlsZXMgY2hh bmdlZCwgNyBpbnNlcnRpb25zKCspCgpkaWZmIC0tZ2l0IGEvbGliYXZmb3JtYXQvZmxhY2RlYy5j IGIvbGliYXZmb3JtYXQvZmxhY2RlYy5jCmluZGV4IDNjMzE3YWNhZWUuLmI0YmZiYzRmODUgMTAw NjQ0Ci0tLSBhL2xpYmF2Zm9ybWF0L2ZsYWNkZWMuYworKysgYi9saWJhdmZvcm1hdC9mbGFjZGVj LmMKQEAgLTMzLDYgKzMzLDcgQEAKICNpbmNsdWRlICJyZXBsYXlnYWluLmgiCiAKICNkZWZpbmUg U0VFS1BPSU5UX1NJWkUgMTgKKyNkZWZpbmUgTUQ1X0JZVEVfU0laRSAxNgogCiB0eXBlZGVmIHN0 cnVjdCBGTEFDRGVjQ29udGV4dCB7CiAgICAgRkZSYXdEZW11eGVyQ29udGV4dCByYXdjdHg7CkBA IC0xMDksNiArMTEwLDcgQEAgc3RhdGljIGludCBmbGFjX3JlYWRfaGVhZGVyKEFWRm9ybWF0Q29u dGV4dCAqcykKICAgICAgICAgaWYgKG1ldGFkYXRhX3R5cGUgPT0gRkxBQ19NRVRBREFUQV9UWVBF X1NUUkVBTUlORk8pIHsKICAgICAgICAgICAgIHVpbnQzMl90IHNhbXBsZXJhdGU7CiAgICAgICAg ICAgICB1aW50NjRfdCBzYW1wbGVzOworICAgICAgICAgICAgY2hhciBtZDVoZXhbTUQ1X0JZVEVf U0laRSoyKzFdOyAvLyBoZXggcmVwcmVzZW50YXRpb24gcGx1cyBudWxsIHRlcm1pbmF0b3IKIAog ICAgICAgICAgICAgLyogU1RSRUFNSU5GTyBjYW4gb25seSBvY2N1ciBvbmNlICovCiAgICAgICAg ICAgICBpZiAoZm91bmRfc3RyZWFtaW5mbykgewpAQCAtMTMzLDYgKzEzNSwxMCBAQCBzdGF0aWMg aW50IGZsYWNfcmVhZF9oZWFkZXIoQVZGb3JtYXRDb250ZXh0ICpzKQogICAgICAgICAgICAgICAg IGlmIChzYW1wbGVzID4gMCkKICAgICAgICAgICAgICAgICAgICAgc3QtPmR1cmF0aW9uID0gc2Ft cGxlczsKICAgICAgICAgICAgIH0KKworICAgICAgICAgICAgZmZfZGF0YV90b19oZXgobWQ1aGV4 LCBzdC0+Y29kZWNwYXItPmV4dHJhZGF0YSsxOCwgTUQ1X0JZVEVfU0laRSwgMSAvKiBsb3dlcmNh c2UgKi8pOworICAgICAgICAgICAgbWQ1aGV4W3NpemVvZihtZDVoZXgpLTFdID0gJ1wwJzsKKyAg ICAgICAgICAgIGF2X2RpY3Rfc2V0KCZzLT5tZXRhZGF0YSwgImluZm86c2FtcGxlc19tZDUiLCBt ZDVoZXgsIDApOwogICAgICAgICB9IGVsc2UgaWYgKG1ldGFkYXRhX3R5cGUgPT0gRkxBQ19NRVRB REFUQV9UWVBFX0NVRVNIRUVUKSB7CiAgICAgICAgICAgICB1aW50OF90IGlzcmNbMTNdOwogICAg ICAgICAgICAgdWludDY0X3Qgc3RhcnQ7CmRpZmYgLS1naXQgYS90ZXN0cy9yZWYvZmF0ZS9jb3Zl ci1hcnQtZmxhYy1yZW11eCBiL3Rlc3RzL3JlZi9mYXRlL2NvdmVyLWFydC1mbGFjLXJlbXV4Cmlu ZGV4IGZhOTE5NzU4ODEuLjg1N2YxYTE1YWQgMTAwNjQ0Ci0tLSBhL3Rlc3RzL3JlZi9mYXRlL2Nv dmVyLWFydC1mbGFjLXJlbXV4CisrKyBiL3Rlc3RzL3JlZi9mYXRlL2NvdmVyLWFydC1mbGFjLXJl bXV4CkBAIC05MCw2ICs5MCw3IEBAIFRBRzpjb21tZW50PVB1Ymxpc2hlci9TdHVkaW8gbG9nb3R5 cGUKIFRBRzp0aXRsZT1XaGl0ZSBLaW5nIEdyYW51bGF0ZWQgU29hcAogWy9TVFJFQU1dCiBbRk9S TUFUXQorVEFHOmluZm86c2FtcGxlc19tZDU9NDk2MjA2NzA1ZjIyMmY5YTYzYmYyM2RjODc0ZDlk NzEKIFRBRzptYWpvcl9icmFuZD1NNEEgCiBUQUc6bWlub3JfdmVyc2lvbj0wCiBUQUc6Y29tcGF0 aWJsZV9icmFuZHM9TTRBIG1wNDJpc29tCi0tIAoyLjM5LjUKCg== --=-ZIMt7PTEoCcrNNeMnaCw Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Content-Disposition: inline _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org https://ffmpeg.org/mailman/listinfo/ffmpeg-devel To unsubscribe, visit link above, or email ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe". --=-ZIMt7PTEoCcrNNeMnaCw--