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 ESMTP id 955A743F8C for ; Wed, 23 Nov 2022 04:15:36 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 60D0568BB11; Wed, 23 Nov 2022 06:15:32 +0200 (EET) Received: from NAM11-CO1-obe.outbound.protection.outlook.com (mail-co1nam11olkn2064.outbound.protection.outlook.com [40.92.18.64]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 936056806F5 for ; Wed, 23 Nov 2022 06:15:25 +0200 (EET) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=YVz+sh4y4Qtc5iK0K+7TaxjQ+CirifF/apSVQUPLz03ZtD10DeU2wm2qC6fKMViXZpuWIWC8KrGFXzrm9oCBqE1fp748TP5k3cp4yvWcHqrY4bd56RGoNADnMUZ6nJG7euEAc6WA3U+DcCkgpZIzcXwgj2/gPf+vjL2BN9kQFzrjl+j3VV0qSCi5IKaUbcbjbxcrZAiCkI0QvNsnSNKWCO2zXo7hNcQw1Y5raPNPfSAWDGWDVQJhLdYrtvv4jUrzsM/d/VoY7PG2kVwy4hPlZza9kiPpzcrA0GcplIsHo0wLP1kg/smJ1eWLUgF7vR4NJsVuPfRrcpAjzckReSchOA== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector9901; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-AntiSpam-MessageData-ChunkCount:X-MS-Exchange-AntiSpam-MessageData-0:X-MS-Exchange-AntiSpam-MessageData-1; bh=9SnI6Vc2DEwU9aDFF+B8xfXInopXyqQqS729qet7Flw=; b=BmWrBdpkgl3mXs3dXxvRh7NKc5ICg/j7Nz1zRCgXemPuDEB5m45NneIyxcG1iTmWISIyUVA0w+dmp+YRLK/fMIveb7EFiiRushl4D3pAg+2AiIU2bsO5MCLYUIH1oo1EjMFulvvX/WZiytT+QbEcZAN6BXX4BASxnoBWbjYitHZuB3pHW3Q/HEBCZTVQXD3NTkk58PC2a6i3NYbko82TCcpoKtF8UYZQQfGkeYYgDVknllzJZ/DiCjydTxHmxklK2TRDMnSZXrzy/m9Y46qkOBQnyuDd1GK0uDIus+/FJ7Zzhou6GR6UKzY3p7M2iY4NwqkXuzn9Vf9k2D6aUgTsXw== ARC-Authentication-Results: i=1; mx.microsoft.com 1; spf=none; dmarc=none; dkim=none; arc=none DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=hotmail.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=9SnI6Vc2DEwU9aDFF+B8xfXInopXyqQqS729qet7Flw=; b=CNdcXm2fKGyx+GXUM+0lbtCbC6go64wZoxJchQYlY4jz7M80KkW9ZKtEAOOEXkprGbyri0LiIecoIoweFO0dOAPbAP1oG2tr4WA2Gc5hsKmPDdIP7m9s5k7xX3q+BhDKxNZxFQFxbUTJHgwfzqlfyJhP5n74iyZDyZlmI128bDIzTfQZms2buLtbyu7Ic62txML74JkjfjG4iFr4M1FCmcfGxP6BTKVQuufwNfRQi6rZHCAawbMC40YhNXxr0nrOfCZDl+dIoT7v8I6VpSwcOSCTZZofM4INaLscf6OsR4/ZTCfZR4zBB7nYJ80svnKMRp2UsJSyYmWbwrwojx3bbg== Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM (2603:10b6:8:b::20) by SN7P223MB0696.NAMP223.PROD.OUTLOOK.COM (2603:10b6:806:263::5) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5834.15; Wed, 23 Nov 2022 04:15:22 +0000 Received: from DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0]) by DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM ([fe80::45ea:be80:31df:38c0%9]) with mapi id 15.20.5834.015; Wed, 23 Nov 2022 04:15:22 +0000 From: Soft Works To: Andreas Rheinhardt , FFmpeg development discussions and patches Thread-Topic: [FFmpeg-devel] [PATCH 01/20] avcodec/hevc_sei: Use proper type for NALU type Thread-Index: AQHYjmIf1Eu5zrkDcEqoBh3qklKHrK1ruMDQgOD/UQCAAAo/cA== Date: Wed, 23 Nov 2022 04:15:22 +0000 Message-ID: References: In-Reply-To: Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-tmn: [FQ5/zdZ3EUBP39wuF5T3x1IlVNBC0wyz] x-ms-publictraffictype: Email x-ms-traffictypediagnostic: DM8P223MB0365:EE_|SN7P223MB0696:EE_ x-ms-office365-filtering-correlation-id: b2c15a5f-abd9-48b8-d7b1-08dacd0956db x-ms-exchange-slblob-mailprops: ZILSnhm0P3neBLaiMYV2lB5LSyKKKau5HXziW9C75rHyHf027qw3LAK6NJ0KN0eFqtjtJxBLAPGcTaRltB7EZDptwqm2owO13rK04bstk9vZBFto6F+4TLjZ4D2kzzJQfp9eZmQ/EHx0FIf+w2TwSJ+4GTexJZ/w7IDUXju8RhSfIY25wXuLLxIGwowjvLsgzhEJI1Kz8xPozETy6sTTT+ndqjztFXf8PEEyilR9DoM+6KREBMT9sBGpMGUfuchQxnamqvXt38fIiCTUGj/3X7YlN13k0mp7S9C77K1UNgsq8jXSxRAZ2KIhmaH9fnkONeQzYzQEiQhMTxFchCTkqO6JQMddt8XM6VAZ+Q9S7Zlds6hxmzbqLwNw4AB3YUmMUAk9b0DUlpTF/82pHyYxDi2DakXg2GSZsCgBJQEjRb4llIC8OJn1QeZphDLy9SjTgUtLObEJmqMz4jxZfAVm2pFnr5Q1ASqgPqMHhY9zVTnwLyHm+4hbK4/IVAvmxBcRHv7HBtIhrkwlhxInQM9FLgu0rjsGWXGJJB5gdQ4lwna+0L9jY/Vj2KzR69Hq8dVYEkRQUl5QU9OtU8S/9daAKCmRSaGlPy37AccsWHbEF8b2Lye2jNRN/qNlz9n6UwpX5EnzVcp5fhKtEXwOL6LycoT9zwG15QOT0b/jvabm9GSYNESTxfzLs0s/SFEuLocyhzSS+/kCoR4Jg1yOfbX8cdIYbObhcp2GfvMecxAu8xoE3ct+o2rhYUwOw173LoSZ5PKFnc+BLhA/yB86vfK+IxSZSaSwnXEp x-microsoft-antispam: BCL:0; x-microsoft-antispam-message-info: Y8awXtIr2HAqZlKzMM8QOKu5EnaO4o4UjVgqTVa7bd+/fZuKSS2T+QNU5SZxId7V3ZFevPjBcbzagz9glJjcDJsG7MS4cBcDRiz26FbpD/FtMMaLkAQJN7UD6dyIX4wt6G0vHeP9CBMLymT6LLmK5xjYCLWoFnq9uYxbgR2bqh/qUgOtEME+y3ytTqxjsfCEtuI/l5l9BfPU4LS5Iwq6DxEfxNd4WZNDVr5r0VN+Pnopi+4QkGJwho+7BAIHyLB8dYqzFS5GGrcgYJBntiktaFJWOE260H3MKCt44v2vXmGcr3y9+BQYYPUz5125nJoV8Mu26Dc5XxGLX755PO5QBfjsNWXNCE9Ti9jzYUGeXGuIBTPoOyzuW4vynRMSX/I+pkUjhZLe1FxCoGYVr3BNYzqX+fH6UIOdCJRzVMpAqaCi1x4th7JvgqsTKwO67I9nYXzL82iYK/a5spH3L0CWKQ/aFvof2I5jg242IaMODkFwfhJzIJHKKN2JY65rXe2NE+M+fvt0u9OUDIlL/khVNaNpGn9I5hCUp9N/+eBhB73ZvfcdLfO9g11MDV82HE7Hy5HkokBc5CJikBejMmLaSv0GxsWXEayU9Jjxd8sdq5OICWQQxO3IpPAm7yRb7KDAMfeIQsbWzicPMIkMsD1W8IrS5lo6YmXZ/Bb9TOMYG8g= x-ms-exchange-antispam-messagedata-chunkcount: 1 x-ms-exchange-antispam-messagedata-0: =?utf-8?B?b3NXQjhSREYrT2tPMHlBUk5VaDFkZGY3OWV0MzloRkt3L1BYZXlJT2ZRN2dl?= =?utf-8?B?QS9taGZzSXhrMlFsbmtWR240RGZUZmJlZERBYy9JVG5NYnpqZXpOWCt3WlZ0?= =?utf-8?B?WEc4WmdOVnJiaWRCdmRCMTdHNEhkR0RpMDBaRSttK25UNnRmSjVyYThBNU9C?= =?utf-8?B?S2RoOEx2cWU4OSszYXEwcXRCZXlSQndudFdTajQydHNQeVFEWFJKSTQ0d0gx?= =?utf-8?B?cWVBa3N3SzhsRGRUUzVXRTdOM0wwVUxWeXBWTTAyUWkwckE2TmxkMUR2cWNw?= =?utf-8?B?OEIrMnVXbHFlU3hmT0UyaU9yZzh4OFdzTXorSnNXYndmbGN3UHNNSldDVmIz?= =?utf-8?B?MjlnT3I4TkQ4UDFBbnlJZXB3L1cwNEpRMnp4Sk1Jb1J4dW9kTGJBMHdWbk56?= =?utf-8?B?TjRhN3lhTkVRblQ2MVh0N2t2WFI3SU5KUzRYMFNRSXBHSDNqd09yVC9rVW1q?= =?utf-8?B?STdNYmR3RWR0blNUWVlFZXRVSitXc3JZSW1WRnQzMWxjQTZlUzRzSzRYc25i?= =?utf-8?B?QVlUWUxzejNmMmRhK3cvSXoydHJwRmR1T0IwY3FEU2J5RkJXbHFaUEMrRHc5?= =?utf-8?B?bCtPd3YvaXVSZlkrZnptZG5kbU4zeDZGUmNJeVVGZCtvZWdlTUVrMzAxMmkv?= =?utf-8?B?bDROeG9MRDUvWE9XYndoZG56aVpPbFJiYjdETHE1V3FwNkxEVnY0ME1IY0xt?= =?utf-8?B?aFJ4RUVyWlRPeVpmT1h4bUR3Z2lZUlJ3SnBpT2U4QW9XNkMwZ0NaUnVQMGxm?= =?utf-8?B?RGg5UVdYZlV3UVRpb05wWG8zQ044UnNWZEZJMmNSWWt1VktUNk5ualZ5NktX?= =?utf-8?B?QzdRRy9LUC9obnYydDZ3cXc0NkFGSzZVcXg2cklKZnV4Q2FLanZyK2NMWldB?= =?utf-8?B?Y1ZDVk1mVmZqcmVFQ0ptTDBFY2ZYK2ovaFZFak8rdlRRSE1sV21YZjlmaEVH?= =?utf-8?B?QnR0OWdIbmtFOE83T1RMdzNzYXptdndUVjJObGRCZEIwTXd2K0dHdDYxT1lu?= =?utf-8?B?MUhkN201WUJNQ1JnNXFyYTdNZlByUDNNNUxNcmV2M0dFQ29sdmZwL1dkUzMy?= =?utf-8?B?b0pUR3hDeG1obHFOZldGYldKbXpRWEZYZjV5WDh1QVVqZVFOZ1Rpcm5lN1Vo?= =?utf-8?B?VUZNcTNISFB6NndqWEVuTGhnYm5xNjlHMno0Ty8zREwvZnU0amNFb1EwVzQv?= =?utf-8?B?bml3bFltUEFHRGdRTXRFbVNQaEpCUjZkU2lmTVgwQzY1U25wVi9lWExuUnhE?= =?utf-8?B?S2x3UEc3MHhMRnpkakZyczBMT1VaRzg0Y2ZNbjJhSG00QXZhUTBXVmc2RnJm?= =?utf-8?B?d0RtRVFXRFBKazFQMDU3SlZTRzVMSWxaRHFJV0x4VjlUU3VXeENHTnlYYWlh?= =?utf-8?B?MlBwbXhyRHEzUU91czNHV2d5YUF6VDJLRkh6VHlOVVYrUkVzcm9LaWEwWUw1?= =?utf-8?B?OGJoSDl2Q2hXOUs0VGNUcUlMdmFZaEJvemcySUhETE94SXRTMXE2dXBXQTQ5?= =?utf-8?B?amVWNmsrQTMxUGNHUGJIMGZqVWRSUUZ3VG16RjQvYm5LckNEb0NITFh2c0do?= =?utf-8?B?ZjlaeDRUSnc2YU5pK3hHalRJdWxPTU5Tam0zYkVQb2poRzJpd3FhYzlETE4z?= =?utf-8?B?N2dnb1VJYjFkbS9wcWxyQkV2Wk9GZXJFVkNCM1c3WFFqclhSa0RPNHRZZTdK?= =?utf-8?B?UnlWMkxNY1dla1kvTjIwdUxnMHhzZFcwOE55V0Nsc3ZBVXRlNzdhdEVBPT0=?= MIME-Version: 1.0 X-OriginatorOrg: sct-15-20-4755-11-msonline-outlook-1ff67.templateTenant X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-AuthSource: DM8P223MB0365.NAMP223.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-RMS-PersistedConsumerOrg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-CrossTenant-Network-Message-Id: b2c15a5f-abd9-48b8-d7b1-08dacd0956db X-MS-Exchange-CrossTenant-originalarrivaltime: 23 Nov 2022 04:15:22.1369 (UTC) X-MS-Exchange-CrossTenant-fromentityheader: Hosted X-MS-Exchange-CrossTenant-id: 84df9e7f-e9f6-40af-b435-aaaaaaaaaaaa X-MS-Exchange-CrossTenant-rms-persistedconsumerorg: 00000000-0000-0000-0000-000000000000 X-MS-Exchange-Transport-CrossTenantHeadersStamped: SN7P223MB0696 Subject: Re: [FFmpeg-devel] [PATCH 01/20] avcodec/hevc_sei: Use proper type for NALU type 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 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: > -----Original Message----- > From: Andreas Rheinhardt > Sent: Wednesday, November 23, 2022 4:20 AM > To: Soft Works ; FFmpeg development > discussions and patches > Subject: Re: [FFmpeg-devel] [PATCH 01/20] avcodec/hevc_sei: Use > proper type for NALU type > > Soft Works: > > > > > >> -----Original Message----- > >> From: ffmpeg-devel On Behalf Of > >> Andreas Rheinhardt > >> Sent: Sunday, July 3, 2022 12:21 AM > >> To: ffmpeg-devel@ffmpeg.org > >> Cc: Andreas Rheinhardt > >> Subject: [FFmpeg-devel] [PATCH 01/20] avcodec/hevc_sei: Use proper > >> type for NALU type > >> > >> Signed-off-by: Andreas Rheinhardt > >> --- > > > > Nice! That's helpful for the QSV SEI parsing. The one missing bit > > is the HDR data (AVMasteringDisplayMetadata and > AVContentLightMetadata) > > assignment which still seems to remain in hevcdec. Would it make > sense > > for factor this out as well? > > > > While the H.264 and HEVC syntax and semantics for these coincide, the > HEVC implementation of it is not completely spec-compliant with > respect > to its persistency: The persistence ends when the coded video > sequence > ends and if the new coded video sequence does not have these SEIs, > then > we would need to attach an AVMasteringDisplayMetadata and/or > AVContentLightMetadata just to cancel the earlier values. I just > don't > want to enter this and this includes not extending it to H.264. > Anyway, I have now rebased this patchset on top of master: > https://github.com/mkver/FFmpeg/tree/h2645_sei (with no noteworthy > conflicts; basically the only change is that the second to last > commit > now adds proper newlines to the newly created files). I intend to > apply > it in two days unless there are objections. > > - Andreas Hi, I'm not sure whether you seen the patchset I had submitted meanwhile? It doesn't go as far as yours in a way that I didn't try to merge H264/5 functionality, but now it consequently splits out the bits that are meant for common use. I was able to eliminate all those makefile dependencies which you had criticized (for valid reasons). I hadn't seen any hint from you whether you are intending to proceed with your patchset, that's why I didn't know whether I should have waited. But I don't mind when you apply yours now, I will look into it and adapt. Maybe you find the time to take only a quick look and let me know whether this goes into the right direction? https://github.com/ffstaging/FFmpeg/pull/31/files Thanks, softworkz _______________________________________________ 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".