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 F22F443BED for ; Fri, 22 Jul 2022 17:58:17 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id AF64C68B6E8; Fri, 22 Jul 2022 20:58:14 +0300 (EEST) Received: from EUR04-DB3-obe.outbound.protection.outlook.com (mail-oln040092074012.outbound.protection.outlook.com [40.92.74.12]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7C92E68B462 for ; Fri, 22 Jul 2022 20:58:08 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=KxVs+N3/U5mvXvAyXpbZivkn5EwNm9s/PTEVw6vYmKv1EVgs38Lu4DFuy0iOOH4uZtOMQ6hyu3niWJRQsJV9w8EIdR1MRGldS6ITUC+IsJ+IchTOPls+YN2vkDGpzgDYOoDd5ZOW9ZzgRZqqfGrX+13zzos7d2A0KVC9YWhVlqBw1AsIrEhL5AJ20zWQ7cuZc1xMkvf48HpJ5ZaY581ZeqK1/16HaUpNUM6V+U4XFfcJMRi7+5lgw6wL6BdWFg7ga7lmAEqVYyyfvWPqOt2HODlJRGxVM4e/jbNfclOnzuP5aarFYWhOljlSGL/FWvq0d/qIylKETXpH/vmNBBHq5Q== 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=wyj7bfPVgrHsaFdcCNcZ9QhdC9yHHmNnzX66vxH0eWI=; b=jMvEF42iuFffFOhJ5aPe1dQbi5fYh3L4QkIpEmFdjMG76+1tY++vMpBag+bMb6vRbufaZnVjVXf+WxDFfU/ip1BC37gBxkldsDUy5fC3pJkWFR5sbXJ7BAZVP5+LognHNoKZRmYGy/S3vL4/rN80ZwB5jkDeq8S16BQd+PwvoA9QG4H5i1nUXH/1NGfNcxFY9pvoOpMsWI4XWa6RyLvcK3VEHzR431wM9cQXmL41wYputkveNS1WN+GdsDKsY2ifm3l04FMlnlt9CW0mMH9UrWZRIKF6VmvOUQwtJzF5gR8M6J48S9kvnvgV9wui1U1bTe+CwDZh5/uBRQ07buzooA== 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=wyj7bfPVgrHsaFdcCNcZ9QhdC9yHHmNnzX66vxH0eWI=; b=M8Kcyjw8uRtI/RY+esTxUrNzrJJeg06u57SxaPnthj4s8KqfW0xvKEdvhMzSuZJT77QtqVDQBDK6C8BInP0XH/zIgtXp1M79CEG9wL7dRgbqUZgRGaGQCtBeDJadO7cZKMhYtZjXaaKyilOBOdNVcAK9lR82CrY6uNdHUvZ/wjYK5aaacjzM2xX7VKuUMD47y/1q7eM/TI9VIIg7fLp6gJ1HQDj7cjHe/omrQHQo9bpEz4ent3CMqpwTz0aRS5bJXJnKGKawvT8f6IMuJX9fSot86g2bjkoeJJPchevO2yuegYixCbSPLNJwRMnKd6FE2+uL/jcYpCYN9nYbsEED2A== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by VI1PR01MB3903.eurprd01.prod.exchangelabs.com (2603:10a6:802:65::31) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5438.14; Fri, 22 Jul 2022 17:58:06 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::210e:b627:bcc9:8c46%11]) with mapi id 15.20.5438.026; Fri, 22 Jul 2022 17:58:06 +0000 Message-ID: Date: Fri, 22 Jul 2022 19:58:04 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220616195534.5278-1-anton@khirnov.net> <20220616195534.5278-21-anton@khirnov.net> <165850434814.15471.8515615540504966051@lain.khirnov.net> From: Andreas Rheinhardt In-Reply-To: <165850434814.15471.8515615540504966051@lain.khirnov.net> X-TMN: [g7j5dANsQQ5h+RQeAkY0zo7oF2wpwSRl] X-ClientProxiedBy: AS9PR04CA0137.eurprd04.prod.outlook.com (2603:10a6:20b:48a::12) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <1e3b5292-c273-6d56-0ab5-588a4a4d26d2@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 29526801-3628-45e9-62bb-08da6c0bbace X-MS-TrafficTypeDiagnostic: VI1PR01MB3903:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: mu0SDDUkNCQtK5/mcf6XI/SjBdXeDR1fSVVNdi4qf+3tQFZaK/RR4/Bt4VkyLwSrknjfwsuvTygDRVSZ0L5PQXCXYGO/u3PdWuG+FUNz+RKvCq9Bq2Q8pP9TxYxtN/7nlnMnRjaz8fEB86g+8RclPZ3Wy62Nfjq4LepAnzqpesSQEEZ3qSHo/lq7FYDUQ6s1o8w2IuLd6wI5tyrVRrYdHZmUuEjs9PSA9KNzCsv5dFfmcix9h5U+Iue4oOcWbygLS8HxFudXeFsHS23qcagXRqZoOPLaJi1Qw4SAkVdlMpYNHRUn7Qgh11ITGJhpJmsBXNXRyHtktqR/Ja7+VGEypCWaSH/RwXAKM2RUYCA6pT7U713RKybUS70Ou7TvO+SPxc0hzKA/+MgrqOvg25eY9m3tTvo5GdcxXM7fdkJFHmVkf2mmSmeH4tyyZuZamjvMJz7AJ8Yc3gWHIS5AGaVSMlzEA4gdVc6O4mqgaI0DtffC/SFByeSG6fGz+kHjIEYe6LpvBdgQPEI4/eDjU/xqleRKFM7QBX4t6AEORNQQXtJI+JoMb1FjMh59kVqUdaFSAABxbVci9k5cr7TvIMiYNCWysaBjQNIV1kDJS8hQxljlUVBtoUO0T07Bjxyze79d45mEvxFbEYQNSgYJUog8S7F5X8Or9TwcHmTolBF0qsNxW6qMv7nrvNdNmuF0ijdp X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?WjhGbkhValFHTURiOTNxbXUvbGVYT2lGVDlZUDRPNnRmYVkwVGQ5bGR5UFFn?= =?utf-8?B?WWROZjM2dXdtZVgyeXFZMS90OHF0R1dVdFZXQnI3b0VCT0VLbmFzNnJwa3lW?= =?utf-8?B?MTNIbjlDcXg0aUV1cDVvQ1EwVEJQcFFkb0x5RHhraU1KUWFnSnZEZWZucHp2?= =?utf-8?B?bVZQdWFrK3pqNFVtMFdNdmtMcUNuZ3o0TDA2SGpIVzZSRkVTSVZkdkt4T0ZO?= =?utf-8?B?MmtlaG5INnJnT3NQczhybmc4SCtFbXMvbGJ6dkh6OEtIYWRxUzVOaURGeUYv?= =?utf-8?B?WFdPRm1aeTVPcWJ0MTZ5RDFkWjBPNUM5U04xYXFuNDR1cjVmU0xDbFZnVm53?= =?utf-8?B?aHFyZVdPUXVsOTRvenRtOFdsdVdGUWR6b1F4TG84eDloVWFRMjNKV2Vod2oy?= =?utf-8?B?eWJyVUljWWJEcGhyM0xBaFF2R1hrakkvUXlwSzAzb3V6Z2d0MnNsTHAwWXB5?= =?utf-8?B?MjBDaDNhVXRrNEdoRkVWQUNyNVdoWDltQzlhM0RIK0dGdFBiT2dzYzRJV3BV?= =?utf-8?B?VG1EOWdxaUFHeGpZR2ZuWGl4am1xSWV6ckJONzVtR1JQTVYwQ1RsZFRSczFq?= =?utf-8?B?WklKSHM4SndXR3ZiZi8xWUxXd3pLR1BYczJJd2dqVUhSOUtrTEdiaHBMZWV0?= =?utf-8?B?TGw0SGRuT3h1K2xwU2h0a3BsYTVlekdsWm81MGZyWGhqWGtrUG90UXlmZklS?= =?utf-8?B?Ky9SemxOS2oxQVI1MU82dk5mTC9XdHVpZmNtYVZKZzVUOGtxSU1XOWZaSWdL?= =?utf-8?B?M1d5Ylp1OWR3U3MyQlFCd0h0ZlR6Q3BrN0pZOVN0NjgyR2pVY3Z4QVRTUnda?= =?utf-8?B?TERhSnNoWWN1UWNKZXZlUThDQ29CUXZmRFc4QytPSGN6cmtFbExVSE45M2dE?= =?utf-8?B?cmI4ZFNrMWpmVDBnNjRFbkxGb1FRY2Y0cm5XRTJ6YzJyeXorZmlPK0lhVHRC?= =?utf-8?B?TUNXRUlmRlp4WDgwWXF2bGVSVkw2ckF2UVAvcVR2RFBnaFhDYVVkSnFMUHJx?= =?utf-8?B?MTk3K1FtVmlRT0FJQmxJb0pLeDFkbnJmMHdNS01yU0RDMys4WVVUbWtwYUdC?= =?utf-8?B?RTBDWjdmemcvL1hzUklYMUFMOVp6M1RISWlQRlA2dE5EeFQ1dTA2amxxQ1dF?= =?utf-8?B?TVhwekZrMkZ5SjNNeVFQOHBpREEwWXVaMVBWV1NoSy9KZXIxeXZjRjBmZjZJ?= =?utf-8?B?QXQyU1lMN1c4ZHlHc2VvMmhUSjFSMTJYUStaR0RqZ2hYZWRQc2JQdWtpVVFL?= =?utf-8?B?MGRkZmxobkV3ZzJnUm1iRytmaHlmVWIzSzJSdnpudURMNDZpYUpJV3lXWkNr?= =?utf-8?B?QkYybHE2eVNGREtocy8vc3g0UEVNU2I4QndEVEVsUDlGZkhOMlI4ajMrdHdK?= =?utf-8?B?bDlwM1VrQ2w5VW8vSVgzVVIzMjJnZDJFa0RZbDFIdDVkN1hXRU5tN1cxVldW?= =?utf-8?B?NVJVZjIrYlNTOW1zSk91ZXdSZ1p3NXBRNHRZeVB4dWF1YzVlTlBjVm5Mcnlk?= =?utf-8?B?WmthcEN3eU9WMWdZTkxxYVBMbnRQVkhWeCtEL1dVaFVhTDNWT092T0wyM1da?= =?utf-8?B?bEI3NG4wY05MMHNsNnJ3RHVwcDJGSWh4UnFxaGpuaktHQVM3MmNFVGF2YWlq?= =?utf-8?B?eUI1WElDSGpzSnhuMmtIb1g2KzJCSXl6Y1FsOFhEMnlLMkxzZ0ZiN0ZEZkUv?= =?utf-8?B?TWY3N2JjUXlVZEpOMXM0RS9GNFVHTCtOUkdBNWhYa1JFMTlscGxtMTBpVEdV?= =?utf-8?Q?r9QF7vDxt6crkU+yRE=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 29526801-3628-45e9-62bb-08da6c0bbace X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 22 Jul 2022 17:58:06.4155 (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: VI1PR01MB3903 Subject: Re: [FFmpeg-devel] [PATCH 22/35] fftools: add an object pool 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: Anton Khirnov: > Hi, > Quoting Andreas Rheinhardt (2022-06-16 23:41:36) >> AVFifos are often used with non-POD elements that need custom init, >> reset (unref) and free callbacks (in addition to the move callbacks >> already supported). So why not add it to AVFifo? The only drawback to >> this that I see is that the pool could not be shared among multiple >> AVFifos, but apart from that it should support the usecases that you >> propose and do so in a way that avoids having to drain the fifos >> manually when freeing it. > > Since neither of us seems to have much time to deal with this right now > --- would you object to me pushing this set as is (Michael confirmed on > IRC he is done testing it) and leaving the objpool question for later? > Objpool is fully private, so it can always be changed or removed at > will. > I don't object to this approach. Sorry for not having time to do any testing/reviewing. - 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".