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 18A4F44194 for ; Wed, 31 Aug 2022 03:24:09 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id EF1CE68BB15; Wed, 31 Aug 2022 06:24:06 +0300 (EEST) Received: from EUR02-AM5-obe.outbound.protection.outlook.com (mail-oln040092067063.outbound.protection.outlook.com [40.92.67.63]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 0F6D468BAE7 for ; Wed, 31 Aug 2022 06:24:01 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=iF3jy8pX3QiIMmwwqt6VuuxAaev89FhU4ByBvuoE8JFOAtYV5r0Nooisghe1eCyz1y6X3qep0LrL9TsBOF0PvQl/6DoNWXytZz08+Ci6aEiKU0uOklSnR8bIOu4Xz7QhyTWwST8uc73lMdKbNhe+OaoSjhzllB6DmIfQVCYHXjqqtRdJPkbamijWJe2pXzCzgdkwEHr/fizPlLxpr8t/3fU775xH0CoSvBVywoRz+CFuzrtecEbwRk0XvOEP3o8coiRbxWpZ3c4pAdwRDMM/yBTlGyo/Q4YwV1M2rBGJz0rgimJkZXNSZp61fD8UvCwROU4MLfuhSQHftTgiKOtLFQ== 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=+5SE7Tm5TAr5/gtDs3J0e22OLahqZuoJ52+B5/LnOYc=; b=X6wipTcGvrNr24nzIt5hW+JfSx0H6G9F47JSYmJK86T/KEFMdYVk+vxLjr3jyEyAXOrFjSflEd8j4lHkFKJLrIBQuEDHHToRnUCuFIdyGhmGQLFpEXcipRRK22FIOYqt+YAM0LG1h/Xnqx0HR+oy8KF57PD5ux2QHWAJd5KJ6L6E5n2kXcyQ1jTB+A/mRUOb2x/z9Okx67ZfHdj2kw+3cUr9ca3aHQNEc/xOKxO88ekT/D/ZUIF61gywdWVLhrAIDqPSexmvU3dGCZL3AvENZL3QtMH7aESfEZDG0XFzRwnvYvDjbC/+ZltlqbjvECg27eBeht/WDAVUVwN7jr/mBw== 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=+5SE7Tm5TAr5/gtDs3J0e22OLahqZuoJ52+B5/LnOYc=; b=OB1nH/6MrlVjKIhvZoOINhSbmM8Jfvv5JgQxuk3qi7MapIZdU3C49YWWuVL3wUyFTmOaOgsEOAIfh17bSgNinJgCxvbqOAZblzmXynyAv2z8LhLz7z29tlUIWVRM56MlTnvth2wiG8pFmUvz8odPcGqpGvF+CFgIWz14WQ+AyLvTaB1VTi8LTI4Wi4UxV8d5jwcCUwEhFh16pHc4Y38eq9qzmxYjH1Pk/hs4yY7DhpE8sfpSSro4K3griFBpyov92qkELaqZkCGVhJaojjuDwYjgcAsZ/nggZGj8SDkerP0K38U8XXI89M6/nY5Pk5N5hcylCymrsoVtSlCfzXik1A== Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) by DU2P250MB0096.EURP250.PROD.OUTLOOK.COM (2603:10a6:10:271::13) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5566.21; Wed, 31 Aug 2022 03:23:59 +0000 Received: from GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::90cc:8832:55f1:c7d6]) by GV1P250MB0737.EURP250.PROD.OUTLOOK.COM ([fe80::90cc:8832:55f1:c7d6%7]) with mapi id 15.20.5588.010; Wed, 31 Aug 2022 03:23:59 +0000 Message-ID: Date: Wed, 31 Aug 2022 05:23:57 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <20220824151828.24218-1-george@nsup.org> <5520bdbb-0f14-8409-32c6-92c14fb8e8eb@gmail.com> <016b2b74-2f94-e2e6-e1cc-e0bcb409c162@gmail.com> From: Andreas Rheinhardt In-Reply-To: <016b2b74-2f94-e2e6-e1cc-e0bcb409c162@gmail.com> X-TMN: [1641j6V+x/PzZl5zMplDIyyKMHS6uWUE] X-ClientProxiedBy: ZR0P278CA0082.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:22::15) To GV1P250MB0737.EURP250.PROD.OUTLOOK.COM (2603:10a6:150:8e::17) X-Microsoft-Original-Message-ID: <0f6aa66c-e271-274f-ab1c-b76f0441fa7d@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: d8a1678d-e816-43ff-463b-08da8b003e7e X-MS-TrafficTypeDiagnostic: DU2P250MB0096:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: EsaUb0z1l+sxbXBhYWUqrmYsnXkoKW6n/h/a5QRHkq1H3eFTtf6cLdjZQAc1/5Q+kDbkNPesZsv6zUz2NSi6fbb65AWh3NygkB6iWvo/WI32m6qxPJyxu3d4qAzDo0X0bQF4/BvDxxxvEcnsKeodd0IketBT5MZ9NmKAGMwx2EYDtanbOI3jlfgWQcSitej1PHOMkWBw63ZymMWH+cmhBxivDqs1HxNpQKVlQ7l1C/ZvUfl2yABN+l2kKK9S6OvUKeo/aIhwsGV7wrzVrWf9AwJ0kKiCVlVbxEZKL3aQKmImMx5H+WjsasL2QVb1v4XTSwfqgczD2EdM/AIAO60oN1UVNKjJlSPpsOGm3bHf2mWPoD6ssVVbMFH8qVw4fbw58cR97NIWmuFbJ0TefYF+oZXDtgwY3kfyoPPEDSgztlfFt8SkSXQo+xJRgHEbeSeISnXVRovg0mv7ihGtLW91o4RKB8fFsYgO5nQFzVgofd7QEwjh/jhgCL49ItU+oeLa3gQmTIbLvpZiCqzq02f1p/CX0GG7t7LNEnRrq0+BYkkr0Rbzd6xq0njuXQmA3cb1y0jXrjQat2O8pMV0aehufcUlEL8qPfAR0yALb/ebZkRvW5GX4cg6mY5eX1KZddelIHJmY4YVksWRCWRhwKvTjVTXU4ei5O8qxcNPjeBQOyX+uZnRizC97haL8TKO13zy X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?SmNzV3JNblQrMlNUYm9zTm9VUGxjVjByQldYMExVUThZSkJkV3VwaktTWk0r?= =?utf-8?B?WmpybUxBSU5vaWdCRUVZb0FWM3krSUh2cG9vbkE0Z05aTkowNUdWNnFiMWFN?= =?utf-8?B?dDJ4QXpGY2xQUjV3Q0pRZ2xHN0NuajhwVTFmSDljdEhNRm9yOGlrS1JZSlk3?= =?utf-8?B?UlkzOUk5WjVrMGVFY1BJZFFBQVVtM0RSNXR4RDFqQTVQUnU4b3FEblpZamJR?= =?utf-8?B?ZmVmMjBuczdpTnVWZ1Q0Y1B0U09PNXdWTXhXd2U5VERISG93N2xYdlVOV2V1?= =?utf-8?B?alJUdnJ4Z0Z5QWxNc0Jkd3FiUGNVdWgzWWQ5REVLV01SdlRqVDl4SEdML1l4?= =?utf-8?B?ajhDTGxUcVozd0FZMHZ2aDRzdHpSTGV6clVDd2xQSnVpcitjb1Z4MFUzcGpG?= =?utf-8?B?YTdRdW5nQXQ1RStQVlNTcjIyVlRyR1F0MllvVjYya1ozMWh3T1pjYzRZbVNk?= =?utf-8?B?MkJnSXV3UWdyMHpCUU1TTGFYUTN6QTFpdTRpcnhjdEQwTmF0SkxabFJraXVX?= =?utf-8?B?ajRka3dnV0hYclExWDVjaUJrOHdQR2c0bm9XNzhucEg5Qm9yUmFQdmpMejFN?= =?utf-8?B?aDJTVjRGZHRxV0FXZEtuRmxMR2M0d2xIcktJYzFjS1FSTGpzc25mNzdwYUpk?= =?utf-8?B?bzdhN0xHRlJhNUFYY3ZyRkdrWGwzRmpOTXNuWnh3ZHF6Mzl5NnBlMkhta3Rn?= =?utf-8?B?UGx2THNUM0g3OWRCRFZYRXIzMm9icko1ZXFIOGNvZTYrYjlGZmFhVXJidi9v?= =?utf-8?B?NjQ5ZEdCS0grMzhiS3BNSjl3eVhyOGx5NkVlSmpVV2ZUTFkvblBFTFNBWVNV?= =?utf-8?B?aXBjc3ZOanpqd3hQS1FtZnNlNDRMeVJ5RU5LczRid1F6Vkp1RlFjR1BuTlpQ?= =?utf-8?B?K0VUYVUvSFVvYjFtN0VJbEd4dC9mRVVSOTRTWGFWYXZ0b0lVaVF0cElUSEZy?= =?utf-8?B?TWZINzdDVVJXMVB4cUtFQ3ppTGR2ZlRCYlQ5Q254LytybmI2cnpjZjVRNllB?= =?utf-8?B?VXpkdEgzbTAzM0hES1hLdmV3M3Z4dzI2RVNiKzN5cFJCSnlLbC9pQ1kyMnRS?= =?utf-8?B?UEZjYjJ1aUp4SVFpbSs1UllEd1JrWi9SYW9lSWpxTzFUejEwOEN2c21uMU9B?= =?utf-8?B?TVVqSUNiUCt1dWFxeGRZdWFORFFRTWFUMkZrVndHRkxMUDB3LzN3M2hKZnBK?= =?utf-8?B?T3dydTlUR2Fsc01MSnRhdUFoaXpWRnNyNmhnVW9xMnFXazY2YjdlWGQ4U0VD?= =?utf-8?B?bmx3M2lYNmNiR2tNS1JjSk1CRTBoZ2xOV3RnMGpCbWlXN1BiTVZOZEZvY1U3?= =?utf-8?B?ck9QUkVXRWsydUphNTRwaHZHTWh5Sm4zZjUvTm9zSnBUQWYxa29lMlAySE9J?= =?utf-8?B?bSsrSXFLVnUrZ2dwMzJ3MmozRld4ZUJVYytUNjA5cElYbTU4SU9UTXc5VFVv?= =?utf-8?B?NU5ReEhsQ09vVHUvd0lEbThLMVBnN05aeHNJdzVGejBHbHJkbnp0dmwyMGZZ?= =?utf-8?B?Y0xvZ2ZHVy9iejlrTzlEaDZpanY0N0hhc2xLQU9nRW9aYUp0V05OR21rWmxD?= =?utf-8?B?dmduU1U2V25kWUszMjh5UkdNcW9BcEt6UzN4NldvQkF4MzVjdVdVVUF1czNn?= =?utf-8?B?cTNaR0NmYm1hZ3VBMG45ekRuSEhUWFd0YVJBL2dxejlHb2kwR3lXRHJPYUZh?= =?utf-8?B?VkorbDZsVjdjclVVdTVNU0x4cEN5NitOYjdES2pydC9ZV0w5ZmsxSXJHNmZG?= =?utf-8?Q?B7Xc407H8jZMr6RVcg=3D?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: d8a1678d-e816-43ff-463b-08da8b003e7e X-MS-Exchange-CrossTenant-AuthSource: GV1P250MB0737.EURP250.PROD.OUTLOOK.COM X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 31 Aug 2022 03:23:59.2944 (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: DU2P250MB0096 Subject: Re: [FFmpeg-devel] [PATCH] lavu: header and documentation for AVWriter 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: Leo Izen: > > On 8/30/22 15:37, Nicolas George wrote: >> Leo Izen (12022-08-30): >>> Is there a reason this is AVWriter wr = foo() and not AVWriter *wr = >>> foo()? >>> Most other APIs return pointers to structs, rather than structs >>> themselves >>> (see: av_packet_alloc). Using a pointer would prevent us from having >>> sizeof(AVWriter) as part of the ABI, as was done with AVPacket. >> >> Yes: to return a pointer, you need somewhere to store the structure. One >> of the point of AVWriter is that you can store it on the stack to avoid >> dynamic allocations when the string is short enough. >> >> Note that AVWriter is exactly two pointers. It will always be two >> pointers, and all the objects that I intend to introduce later will >> always be two pointers: one const pointer for the methods, one pointer >> for the object itself. >> >> This design is essential to the features I promised for AVWriter and for >> later. >> > > I don't see how you are planning on avoiding dynamic allocations by > stack-allocating AVWriter when AVWriter itself only contains two pointers. > > I also don't see how this design is essential to the features you > promised in a way that can't be done by just not making sizeof(AVWriter) > part of the ABI and returning pointers to a struct. > He is not only stack-allocating AVWriter, he also intends to stack-allocate the actual writers like AVDynbufWriter, AVBufWriter (AVWriter is just a wrapper around the underlying writers). This means that no allocations need to be performed for AVBufWriter at all (and due to the inherent small-string optimization of an AVBPrint, it can also be avoided for AVDynbufWriter in lots of cases). If you return pointers to an AVWriter struct, you need to allocate this struct somewhere, which means that your init/av_dynbuf_writer_wrap has an allocation that can fail and therefore needs to be checked; and the struct needs to be freed lateron. - 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".