From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from ffbox0-bg.ffmpeg.org (ffbox0-bg.ffmpeg.org [79.124.17.100]) by master.gitmailbox.com (Postfix) with ESMTPS id 61C064F1DC for ; Mon, 16 Jun 2025 18:27:12 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTP id 8972468DE00; Mon, 16 Jun 2025 21:27:08 +0300 (EEST) Received: from EUR05-AM6-obe.outbound.protection.outlook.com (mail-am6eur05olkn2025.outbound.protection.outlook.com [40.92.91.25]) by ffbox0-bg.ffmpeg.org (Postfix) with ESMTPS id DE38268D22A for ; Mon, 16 Jun 2025 21:27:02 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector10001; d=microsoft.com; cv=none; b=zQQ0HbwDahN9pKgepxxgwaMHLYSz4kQIkY5AwyhxnTH8pUAnZEvCi54whMs3T9uhEyWLwNNUPhEbp530oMWNBss9iddaG1lUXGmYvsXa1vqBUBVTzQplsyfWM3dI24vFBekM5rUXSwV9bZoHtjG1h/Ri+/f0K4Sg5CGVMImy0Aq+llSKiuG/FwnSCEB45U5Tj4GmD4Lsuwj6B+gusr26jpWhQWEbLQ0LqNDgC0N5ggRWEqGtEkN/WpLwHy7u5kDqgQFE+gUQU+j/4MdV51dQ8WSW5Hse6+jsBGLn2GGOnMsES1Xjp5EP30N3lLEopTBCIAhKgpFL43HcBzmN4+4MtQ== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=microsoft.com; s=arcselector10001; 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=MLu0skPtoTeYXxAjKyvlZigTWltRppjoVQ/ZFMNVcrs=; b=rEdQgfCv0CQSznpoZ+jdcvngfDyG31uPk2NKZYvSBHwq/KSVJ337I8lbtp/XVblZKFgcMdmRBNcpVMIO0G38r0FmvOZua4RWtkJVnsvW3HqeebUkrulK57WEmYKFRVX0EdWm84LbPPa5/mbpU17zpE1ZW2Xh4cnWgH4TapScXf2f8mQn1RLMjGNj++BvZgC0MAj/S2egz6Kc59No9cZYS6m5Ja0SjVxp2Fp0MLLqO6v4gt9W6wB/h8QKDbj+rbCyou+3EEVyHlwjTVJQhgNvwHJYhKCC5kj+QWrAMO+rHPpVR8jtOD1AxReZTD4zWjjLCyjeZ5+Dim/vzvPmcd+s+g== 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=outlook.com; s=selector1; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=MLu0skPtoTeYXxAjKyvlZigTWltRppjoVQ/ZFMNVcrs=; b=f4L/qeSAqnT4wQ6XBCVjU9b3dQnS4apqBCy3NexYCy416Fpq83KTtbMxp9zi6NZMEK1+wFO5RR35kJY3D+tc1fjkQVg1ZqY5StNDIvcYClkZ+KahPxqT6hf7g+gjCzkEnSTLxAQgShAMeyPAetzkIE/pvzyHcBrV2D3b+is1G/PQsoDzci1XB8D73w41mym2qKe5RwS8XAePmf/I3VYTWIqtbM8aPSkuLiQhu9EHYdZml9cM+A9em6Z0b1JSOMc1rWKLcmMp6WUXvOUuGXhC+C2Q+5lOP8ZZLr/bC44vls7oxT4OTiz93sXjXJ7cKSYJks9myMbbxMOEhUOEoozSlA== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by PAVP250MB0932.EURP250.PROD.OUTLOOK.COM (2603:10a6:102:307::8) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.8835.26; Mon, 16 Jun 2025 18:27:01 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::d6a1:e3af:a5f1:b614%7]) with mapi id 15.20.8835.027; Mon, 16 Jun 2025 18:27:01 +0000 Message-ID: Date: Mon, 16 Jun 2025 20:26:59 +0200 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20250607213509.16424-1-timo@rothenpieler.org> <57ffc654-1acf-4bb4-8304-e30646ed427d@gmail.com> <224b0f55-d991-4098-a12b-ddd4a85b226b@rothenpieler.org> <7afbb7e5-b32e-4001-aa3f-7cd0d44cd859@rothenpieler.org> <629167d5-1140-426b-bf70-74c998966646@rothenpieler.org> Content-Language: en-US From: Andreas Rheinhardt In-Reply-To: <629167d5-1140-426b-bf70-74c998966646@rothenpieler.org> X-ClientProxiedBy: FR4P281CA0426.DEUP281.PROD.OUTLOOK.COM (2603:10a6:d10:d1::9) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <789f5583-58dc-441f-b21d-b88cfadf97bc@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-TrafficTypeDiagnostic: GV1P250MB0737:EE_|PAVP250MB0932:EE_ X-MS-Office365-Filtering-Correlation-Id: 53fb8314-16d5-410a-3882-08ddad0362b3 X-Microsoft-Antispam: BCL:0; ARA:14566002|5072599009|8060799009|7092599006|6090799003|461199028|19110799006|15080799009|440099028|3412199025|40105399003; X-Microsoft-Antispam-Message-Info: =?utf-8?B?TGNHTi9zL1hNN1hLNVpKQXRlc0htOWFsTks3S2pER3VXN2tKd1hJczUyM1lR?= =?utf-8?B?eVNnUWI4eFE5K2k2YnZiTzBWSjJQWEdmbXNuOVY4cFkyai85NzZzWlNlOEJ2?= =?utf-8?B?L250VW53cVhNLzZpNURPaFhpV0VwaU9pb3JtUXRXNGtvWUczcDBscjZObGVq?= =?utf-8?B?ZzRRQStTd2F2NUJUUFljT1NxS3RNZFpqNW45dDhOTlN1eHFBOXJQdVgrY2RV?= =?utf-8?B?RERxK1ZSUzNsUnBtQ3FrNUhkTkhqazIvQVZEZXZCeGxMUzFvS1o5UnA4Szhp?= =?utf-8?B?amRzakhJeC9RaUplS2xHYTg5OEJ5YVY4OFdUclh4eWJXeStwN3pJUDNvaFcv?= =?utf-8?B?SXZ6TWNDUmpaYjhaMXA2Slo4SjdVNnZRTUFmS3Nsd2RzVnI3amF2SmY3bTkw?= =?utf-8?B?S3BkRkd6Q0JEWGVGd1dWdVEreDE2WlJmVFpzZ3VocGxKalhNK0VPaktQUmtV?= =?utf-8?B?ZzBMY1V2NkpJUjZrcmNtWDZFblpOUUNjZHo2eDRmcVpHWnFHUmNOSEYvUzlH?= =?utf-8?B?RGZHMEk1QjNSYkVPWGFZOU1GM1ArZnlmckp2ZEt2eG80TUVYS2xTQjhacTkv?= =?utf-8?B?bVdzMURzRE1abStJUUN1Wit0VGZsQSsvY2JnWC9VUjF3UFRzdnk4Yi9hQVJV?= =?utf-8?B?V0hhVDJSYm5FZElxQVIxSEpJWjVEU3lKZEtCOVFNS0dyUStQMG05MWFSZFNh?= =?utf-8?B?Q0diL3ZYVlZzREhvUWdCSUJjdU5oZk5HNGxiRGRzV24xeDFkSDBuWlh6ait4?= =?utf-8?B?cTNYMDZEMjB0a0hxa09DWU1vZDVRd3d1WC9WbUpDYmh4QjdOWFJoMXI0SkFy?= =?utf-8?B?bUdQMDJuTVluUVc3V2Q0TFNhRTFvcmUxUUpiYW5ROWQ0SE9MS2x5QWNhcWht?= =?utf-8?B?azR2a1liL3R3aW1WUlpnUFBjdElERVBTYkpFR3ZabkhJUlJHQWhFMmpLR3RF?= =?utf-8?B?dmw2a2ExMWJtNVVXYXIybkg0L3hxUU4yNDlsTkdWTDNTcXg4WkxFL2R2dUlS?= =?utf-8?B?dUtqdXFVMnQ1cG5VNXJaT3FVVjhvN09rQVZVR2ZjU0QvYzdFS2hTS1ppUm1a?= =?utf-8?B?cGd6S1ZwT1Y2SWtGckdObUdadnhFOGV5aElUeTBLeUFvK000eitXYis2L2pR?= =?utf-8?B?dHRmeFZ3ZThwbEhWODg1cUx6ZncycTRubU4zTUhMdzd3NVQyNnpMQURHU3dU?= =?utf-8?B?cXA1R3BhdkVQd0tTRkZZRU5ieC9kcFpxd3kzcFpiOHJ6TUlYQTN0d05QSmtZ?= =?utf-8?B?eGU2eGxlaDRrbnFyZzVrL3M5eEVGOWhPR3NuRUZGRDdUSlU2ZVhkcGd0MzZ2?= =?utf-8?B?U0tMMW5rbDJDbldieFZ6TGg5cUFnUnl3anVQdHJneHNSb3BZSENzdDFQMi95?= =?utf-8?B?VmdZeWxoU1FkMEdsT2JKd2N3NWZXa0FkMHhYaFM4UEdPL1J2OTdnUC9VVGUz?= =?utf-8?B?ckQ4SE9KZmxNNHJid245QTluU0o3Unh0anRPU3RmVnpMODVVT0pCQUEweDQy?= =?utf-8?B?ZFJzUEV5UXZON0ZEOVBjbk1Ubm1kNElBS21iUndOT2duVDBVN1plNm1BWnVL?= =?utf-8?Q?pq19J97TM+Ct4+y7P9FZDQqII=3D?= X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Q3NpR1pVK1JDZGMvOFZTc3RiNjdacnFrT3Rnc0lrVlpIUWxqbXVSK1B0VkRm?= =?utf-8?B?Q0ltNThzV3F6cEo4VWxKQmVpQ2E4RzR4NVlpNm9VY0lONHdQSEhibTlFeDNs?= =?utf-8?B?amcxbXpyZUFkNEYxblk0RlZIY0xIZHcvOVBNMWUxVFpzbzZRNlUxMlplMDkw?= =?utf-8?B?MFBibUFEQlFBb1ZoSlNiekViUzNJblNXVzc5WnM5cHJnd1RoNHp1R2E1dHlX?= =?utf-8?B?NXkzbS9SMkRNdS9VYSt2bmZwQW9uZjdOQkxreE8rNCsvN01tbnRIYU1vNTI4?= =?utf-8?B?ZnBDOXdpMTYyQmlJWHdSdGFvbDVKK2x2SkxWMkxSWmtGemxZN0FHam5lYkN6?= =?utf-8?B?OUlEZ20zTnh5aTBPc2I0OEd5UmZrb0VpMGxRaXlMZ05Oc29YRzZZWm9qWmZk?= =?utf-8?B?QlZnOEdvNko3M1dzcUg2L1NucFNLLzVRSXdQTzY0ZndHWGp4eUtiMGFiRU8v?= =?utf-8?B?bUYva1hlRUcrQmkvc1czUmtySzJKZzFzeDBObVM4NnZobHZGVXU1cTBnL1h3?= =?utf-8?B?TjBVbTlrQkdVdzEza1R4NWd0bkd2T3RUMUJXN2R0djhYM0k2bWxaenNsZUt3?= =?utf-8?B?UGlONFBiR2pIbmtkalo3SG96dzQrbHVIbWJ5dnQyYml5RHZkUXlEeTBGYXdv?= =?utf-8?B?S25UNFN0UkdQTWxEaTJPT29KY2pxTWVSbHJsVXZ1WUNIdGV1N0JVVzdVZ3ZY?= =?utf-8?B?TXhoL3N6RlYvTGt4SkdwSVpMb1BJd0ZMWXl0Y0FGSE9Nc3hXeXlNTXRpay90?= =?utf-8?B?TWdGUVdlNzZqM0JiTVhlVElwcU5RampYRGRlVFd1OHNOSnFFUFZhdzhnMDJT?= =?utf-8?B?OEJQamhLY0lXMHAvalZ5b2dzMlBYKzhHNUQ1OHVkODFmbWZLajNrU2phS2NH?= =?utf-8?B?MWZDVkVhd1BEZzNheFpuNmJmREd1aVMvZDJpTHFSUE9yTzZaRmFpNFZ6QW9L?= =?utf-8?B?MDZ0WklGcktyNGxKMk9CVC9kcnl2amtTaFkyOXVMaXNjY3VucC9yRnc2K3RM?= =?utf-8?B?aWg0bnE5VDV1aXFCamtoNzRCZTF4NXVNWGszdUFmakNlOW9UNWJuYmpsY0hX?= =?utf-8?B?Tmo0Vzh4SG9rdTdjMXlZRk5tVXhZVFdGaXg5S0tiSWpCVk8wSitTcWc1RjVD?= =?utf-8?B?TWFLWDF1YU56UVhxckJ0dTgveTNoajQ1NU4wd2I5RFRGbHUzRDcyTmdrSE9Q?= =?utf-8?B?NmVXOW5kUE5ERHZkRkQxTWcycUljcjdrUjRKZi9vZmIwTmtublVzd3F4cVNX?= =?utf-8?B?SnhUWHE2b1hRSmNIc1BlVjdBUC9tS3hEd1kvNVROR3BiT3NIOXo4T1o4U0VS?= =?utf-8?B?d0taTFAralU4Y0tHNE52eWlwY0VaZGczVWRNSm92dmsyRVNQTnFWSUVvLzZJ?= =?utf-8?B?bVpxaGF5VzY3VDFsY0h0Kzd0NlE3cGFjQXBzK3dhYVVuQkp3eVBIV2dHbGhS?= =?utf-8?B?dVJmbzVNTVJ6UXNZY2dLdG12U0NMUXNvcFQyNk5oRlIzTVVVVzByZ0N6TW1u?= =?utf-8?B?NlZHNHp0SmI4QXdQNHRPMDROMzF1V1hJUXNYd3phSXVPdngyTUM5NFBKUEI1?= =?utf-8?B?YXVFcnlXdWRRSC9yai8rNGJZQjJOZTNqYzI5bkJRY3hRdkJkT3A4QWVjSlll?= =?utf-8?B?MVg0ZGRBeDF5WDkxUGgvQ0JGRzZuM2thZXV5SXk4SGpXb3ZzRkg3emo2eE1n?= =?utf-8?B?RjR2Z1lUMXpxcVJjWnhNTWZOSkZOOVVGRFNpYklKY2d6UnVEdit4bG5GWkYz?= =?utf-8?Q?ng53RorE3HKx4k+nj3TyIko9JC5oFT2CLwG95OB?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 53fb8314-16d5-410a-3882-08ddad0362b3 X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 16 Jun 2025 18:27:01.2236 (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: PAVP250MB0932 Subject: Re: [FFmpeg-devel] [PATCH 1/7] avutil: add an API to handle 3D Reference Displays Information 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: Timo Rothenpieler: > On 16.06.2025 14:55, James Almer wrote: >> On 6/16/2025 9:38 AM, Timo Rothenpieler wrote: >>> On 13/06/2025 16:07, Timo Rothenpieler wrote: >>>> On 10/06/2025 00:09, Andreas Rheinhardt wrote: >>>>> James Almer: >>>>>> On 6/9/2025 5:59 PM, Timo Rothenpieler wrote: >>>>>>> On 08.06.2025 17:45, James Almer wrote: >>>>>>>> On 6/8/2025 11:29 AM, Andreas Rheinhardt wrote: >>>>>>>>> Timo Rothenpieler: >>>>>>>>>> From: James Almer >>>>>>>>>> >>>>>>>>> I don't like that you add another allocator for this; instead >>>>>>>>> we should >>>>>>>>> add a generic allocator for the frame side-data types. >>>>>>>> >>>>>>>> Wont work for packet side data. And i purposely didn't add yet >>>>>>>> another allocator that inserts the result into a frame, like >>>>>>>> there's >>>>>>>> in so many other modules, because eventually the generic one >>>>>>>> would be >>>>>>>> introduced. >>>>>>>> >>>>>>>> You said you wanted to take over my work on the generic allocator, >>>>>>>> but not sure if you did anything with it. The core issue was >>>>>>>> handling >>>>>>>> more complex types that didn't just have an extra nb_blocks >>>>>>>> argument. >>>>>>> >>>>>>> So, what is the conclusion here? >>>>>>> I'd like to push this set if you can come to an agreement. >>>>>>> >>>>>>> I haven't looked into it much, but implementing av_tdrdi_alloc() >>>>>>> in a >>>>>>> generic way does seem a bit hacky. And other types might need even >>>>>>> more info for the allocation. >>>>>> >>>>>> The set LGTM. A custom frame side data allocator does not imply an >>>>>> allocator is required for this struct. Frames are not the only user. >>>>>> >>>>> >>>>> I was thinking about a generic side data allocator (for the >>>>> AVFrameSideDataType stuff, but it is not meant to be AVFrame >>>>> specific), >>>>> so that it can be used by more than just AVFrames. Will write one >>>>> tomorrow. >>>> >>>> Would you be okay to for now pushing this with the API turned into >>>> an internal one, so the public API can then be decided and >>>> implemented later? >>> >>> Will apply soon with the new public APIs turned into an ff_ symbol. >> No, the allocator is required, and an internal function like you >> suggest would need a separate internal header, and will make the >> struct virtually unusable outside lavu for non frame side data users. > > It's only meant as a temporary measure until the final public API is > figured out, so the rest can move on. Sorry for being an asshole and forgetting about this. Will really send a patch for what I intend tomorrow. - Andreas _______________________________________________ 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".