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 87CCF435ED for ; Sat, 18 Jun 2022 17:08:42 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 4EEAB68B5B9; Sat, 18 Jun 2022 20:08:39 +0300 (EEST) Received: from EUR05-DB8-obe.outbound.protection.outlook.com (mail-db8eur05olkn2024.outbound.protection.outlook.com [40.92.89.24]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1A2A168B2A1 for ; Sat, 18 Jun 2022 20:08:33 +0300 (EEST) ARC-Seal: i=1; a=rsa-sha256; s=arcselector9901; d=microsoft.com; cv=none; b=oIYNAaO/cijWODPUR9zWoZdAlwnQSMu7ANOeHuguLzCElrboOGiON6xV3LFXvxbn232Yq0IHkxhNmecB7U+KPFbKP/sAVoZvRNpLkhLI6gOjhgBIZDAnHBLg4Pq7BUrSwt0tDOsWl7a0PKnWISFUpwXr4Hh3JmsJW1Oar6FWKPr2Qacy09N1usPqFrIGngzpMLuuFNOaQaCTSWiE0XnXrKeafhlwPlyzKZbUE/nJOn8bXIk8DGNApA6+/+jgxfMMLU2SJCD6NrK5VUQH+0PIduGTa1C9pPdJa6mUkRwDIjeUAZHMzp/7oMp1wiW3a1CS4ni2p+K2FfH9vX3Z/7R+Tw== 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=opJwvrKU1+6SdTY0CY9qGGnHtBJuRbTyfHw9TewkVWE=; b=je3k2HlIVRBhaXExGN9ME5WaofH0ErGqj6O/VkN/i1/kCvFBpGi+M5qYIme20vj0UNyzU68wiioVHmp4A9r12P8iobeNgdkVOSWuuDC4H+KlwOUiX7xejecRS2dusp1TNsZuX01TkdgnysKHKbGgrd4AGr2VU7hBR8jBZWKqfm2JCs3mVHkexbBf6jjpANkFf6zy5pccIN6azNwlwx0aEGm/tjUtjPduvJpz5avM/dRTzsGrnPtWSmwNaYkm60p0AR0zoRc928Wy/4egofoBtd5M+63mYq//oZXkjuviTxiUHdvi7ofJQHdP92VY1a50sRm+lbGUkKu7EiBRDiOWww== 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=opJwvrKU1+6SdTY0CY9qGGnHtBJuRbTyfHw9TewkVWE=; b=ig94beBeGTJNS+pfc/cXKV1fBdrz2U45By98/7BpszgTVzGg6A2ubc7a5R1YO8mefA8zlz/zm9Lfwr0psup/H30p40eOA3FQF999eRgDdPF91LSSemI+tvLXKr+LI8pycWdckkQTVpD0cx7+JcGjDw7kBP6bIRTBh7OGAx3nDIk8rK3U84u9wZleZcqhT4H7EK8mvPxD8bOZvZz+I7g1eSOFwpYjumucbOxNX1qmL/EdylfMFnQqz2T0/7Nh+AWcNbk956cZnKl7fiJXSaW3UxvGjWc+Rxx1JDJu0fASQ7pADxRnHa8Ly8VSKae4AB7jmhzWRoNZJ2O9EXhMitoGdA== Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) by VI1PR01MB5359.eurprd01.prod.exchangelabs.com (2603:10a6:803:cb::20) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.5353.16; Sat, 18 Jun 2022 17:08:30 +0000 Received: from DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c]) by DB6PR0101MB2214.eurprd01.prod.exchangelabs.com ([fe80::60b9:9f29:40cc:f01c%10]) with mapi id 15.20.5353.019; Sat, 18 Jun 2022 17:08:29 +0000 Message-ID: Date: Sat, 18 Jun 2022 19:08:27 +0200 Content-Language: en-US To: ffmpeg-devel@ffmpeg.org References: <122266341.9130638.1655553986479.ref@mail.yahoo.com> <122266341.9130638.1655553986479@mail.yahoo.com> From: Andreas Rheinhardt In-Reply-To: <122266341.9130638.1655553986479@mail.yahoo.com> X-TMN: [naCtdkKdGEeuju/nTQnhDa8Atg61qDT9] X-ClientProxiedBy: ZR0P278CA0187.CHEP278.PROD.OUTLOOK.COM (2603:10a6:910:44::9) To DB6PR0101MB2214.eurprd01.prod.exchangelabs.com (2603:10a6:4:42::27) X-Microsoft-Original-Message-ID: <9cb9ebb6-3211-7792-b27f-1fb860a2ff08@outlook.com> MIME-Version: 1.0 X-MS-Exchange-MessageSentRepresentingType: 1 X-MS-PublicTrafficType: Email X-MS-Office365-Filtering-Correlation-Id: 2bafb726-62bf-45f8-3f59-08da514d2a6b X-MS-TrafficTypeDiagnostic: VI1PR01MB5359:EE_ X-Microsoft-Antispam: BCL:0; X-Microsoft-Antispam-Message-Info: Kj8OodJJzCAOr/eajeKeoYMX52Q1/LFTjl6dtOnD9RD8nu7Pdina6uB7aVhGoSWgBiT8z4fS1S9Q6AeixTWnZxBy7VQ5zGA45NqZxCNPYnyzhRzMcKzxOr68G0KUjw2nl+wpZIC8zb7K9Ddu/b1bPyCIbG4ZzCTpnv6Nim7y8ReCW0lRPDSAY0F9oi0OSY6UmFvWb+yeerRK2Y9mSzaB/J+zhOLVrGoQa9LX8JWT6FgM5fv5MuycqddITgeJuBJZpUpiIv26aAdumS+ReFswepl3jm1Fq/RLuxlNYeuPOQg8LIXXAh2YUJsylHlGx5927l76hxKw/rD3jOICpInlwEbotFfgLcUrqF0aRHnWmg+TtNvRpFkDeiClZdvDErj+f5acndywZX+R+dFS5ALvu479cayYNMqo5DD9nRb/oMvrsa47dhuF78XMH7Yzn7bu6eXoLicRNrS4If80lBPO9WQTAyKDpWLJqc5HO+2tO80Qz5M2poN3u3m2i5aYit1TRNHk9f5/QmaEzVtBbRuN8/bvKqI0thGG4wNYWJ+Hqf01SOt+FwYwmZWIy7zFCXobkvj43WXT/t3LdPLBpbwLMw== X-MS-Exchange-AntiSpam-MessageData-ChunkCount: 1 X-MS-Exchange-AntiSpam-MessageData-0: =?utf-8?B?Nm8yZHhPQndXTVVkK0F1VWYvWXgwSk9rWlQySDVHUWh5ZVVKcDhRTzNabXl5?= =?utf-8?B?Rk95RHFOL2lsVGp2K050MHlZQzZrTlFlRVJUNlZHTEU3R2tEL2IyRkczOFMy?= =?utf-8?B?UzdMU1hLQ3I5T05JeGU1U3R1ZkROWTZXMTlqSWVEV1BNeDN6dkw0a3F4cDh3?= =?utf-8?B?Nnd1VHVqYlNYVkJaa1pJWmxuRkZ3UDQ4SlRqdFVLVnNvYjA3Unl1QlBQNk95?= =?utf-8?B?YUsyOEd2OTdQcEltTE1uaHU1UWNwakc3VmpOWXNaYnVPY0doRUxtWFB5RHFN?= =?utf-8?B?dVIyUWJkWDVRSmZLSlpLT0N4azZqaExnUmZZdituMWpHdU13dWx6aWlmZlRB?= =?utf-8?B?YzNwRVEzRG54Y1Zqd0gxend3dWdKQ0tNS1h6bS9FQmlDeVozZFZscVdlcmEw?= =?utf-8?B?aGZhSTVzbEJYeW92SmtSdEk5UzZaamdKeVMvdDBpenNMcFE5VDNiVGZiS0JH?= =?utf-8?B?cFBWeVhmZ3lyaHE2SzQrN245N3UzL1pvMVltOVhhbThFRUh0WUkrWjJ2aHd4?= =?utf-8?B?NmxjQnhjMW8ranVhS0huOU5zSDRzazZDSzdkbnRaSlBaWW1LR2JNcEJoekQ3?= =?utf-8?B?bGIvNjdRT0pna1VocG5JajNJaFIzZDk2REZqaDhTbitXc0lnekdXQ05xWDFN?= =?utf-8?B?cDB0bUkzNyswQjYwbzduU1hVV04vUTUrd0p3bTc1SGYyMy9qdW5DazFOcm9Z?= =?utf-8?B?ZFlXR0RhNVE2Z2kvWng3b0pnbmh3TlowcncvSE9Jb1p3R3hoRThZZDF5S013?= =?utf-8?B?RExBdTIvZ2xMaWNOUk0rbGt2dnlQN2tzTkE0azBrQkhnbkFkRlpnVjY2OTBH?= =?utf-8?B?U2dZaExtTWZUV0RWZTNYc1ZwckpFb0lrdGpVTW9HTVJraDMvT2Z6ODVXcnBa?= =?utf-8?B?RC9sLzNiclJOSlFvODlMK2lWa1QwamlWUzNsdWMyZUFGUXJJS0I1UEl5UnhF?= =?utf-8?B?VjVjQzVBNjJIM2l5R1p1c1p6dnkwaytGeDNiTjFIb3NqMXhCVHU1bnRCYlZq?= =?utf-8?B?MitYMGZHN3F5Q3J2SXBlbnhjb2ZOVUtqNGFxR080eG52d3hCYU42MThzQytx?= =?utf-8?B?YkpEY3R1S0FtY1NsdjlZUlRzR3EwQ09TSXFwVVNPL3kyNDRHR3JJU2c2Sm0w?= =?utf-8?B?c09vQXR4OGNab2pNY1VVZFdCeXVUOE1Ea0xScXpjZ2Y2Vlo1UEZORkRmSGd6?= =?utf-8?B?ckwrY0hYZmlMb2VicmhQL1hGdU9zTDRISmF0V2w5dyt0UGI0dC9MNHEyaHE0?= =?utf-8?B?ZGIwNldnRnJOTFhVSXVzQzNzQ1dzTUx0ZUY0SmdrMUFKZUU0QTJTaTViSkp0?= =?utf-8?B?Nlp2SUJ3bUhVWW1PRUV0TmlFb2NVcUhpQ2lkYnpJaURORlA0SzRxS3BzQmpL?= =?utf-8?B?TXN5UW1BOWhydjc4MmYwS3dBOCtlNHRmRGp0UktpNlJKWEUvWS9LL2tqY0pY?= =?utf-8?B?WkxDdytEQmROZzIyQnNqRysyTm1XckZ3U3JBaXNRL1RUaHBwYktYMTJIWFVO?= =?utf-8?B?KytETVBsSTZKeVl3blNYNk4zblAzWXVCd2ZtMmtneGxId0ZjTktDT2I5S3pV?= =?utf-8?B?MFFEQjNRR1ZBTi9LOTJ5MDllRStoZHFkSUdRNGJaQ1hVb0dQWTR1TW1YbDJ2?= =?utf-8?B?Z2VrNjBQNnlzL3ZDdVR0R3QrS2N0UUdSQktodTh5UzZmVXlIME4vKzhsYUU0?= =?utf-8?B?bGp0ZzVIVGdJWHpLcy9PbExwalU0UW9IY2U1UG9uWnN1VzIyU0htanVRTGVu?= =?utf-8?B?aC9yekswd3hXU25OaEJJaE5RQnZ3QlRJcWY0ZDFjK0V5cmVoWkhQOXZFandy?= =?utf-8?B?Ym5rdTNCZnNHakxiMDAxd3VwZGRlMDVSVnhISFFKWnpxaWJtWVJUMUZ4bzRp?= =?utf-8?B?K1VwRzVwS2p2OTBxM0JNaWdqeTRNNkJIRGRxdFBMSEdyaFZHamZIQ05OTkZN?= =?utf-8?B?U2wzRFpFK3JtOXBxVCtQL0N6d3JQNnN1MjF3dDNxbXl4bjUyeW10UlhKQkwy?= =?utf-8?B?enFTVmxmSWh3PT0=?= X-OriginatorOrg: outlook.com X-MS-Exchange-CrossTenant-Network-Message-Id: 2bafb726-62bf-45f8-3f59-08da514d2a6b X-MS-Exchange-CrossTenant-AuthSource: DB6PR0101MB2214.eurprd01.prod.exchangelabs.com X-MS-Exchange-CrossTenant-AuthAs: Internal X-MS-Exchange-CrossTenant-OriginalArrivalTime: 18 Jun 2022 17:08:29.7818 (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: VI1PR01MB5359 Subject: Re: [FFmpeg-devel] [PATCH] doc/examples/muxing: code rewrite with improved readability and fixed issues 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: Paolo Prete: > Please review this. It's a code rewrite of doc/examples/muxing.c which improves readability and fixes issues. > More specifically: > *) Original functions of muxing.c have generic/unclear/ambiguous names and they don't group logically relatedblocks of code: this makes the code hard to read.See for example open_audio/video() and add_stream(), which mix initialization of the encoders' and muxers' stuff.A redundant structure with an ambiguous name ("struct OutputStream"), which is not part of API, is widely used too.The patch uses functions with clearer names and with blocks of code strictly logically related, such as: init_encoder(), init_avframe(), init_muxer(), convert_frame(), encode_frame(), mux_encoded_pkt()... Using a structure for the user's data is actually intentional, because users will probably use one, too. And given that it is the user's structure it is of course not part of the public API. > *) Errors are not always properly managed: in some cases they are not propagated to the main() function and a call to exit(1) is forced, causing memory leaks. This has been fixed as well. > *) The program doesn't work for some extensions (try for example: m4v or mjpeg). In addition, it gives weird results when the filename doesn't have an extension.This has been fixed by checking the filename and supporting a fixed set of extensions. > *) The program claims to output raw images by using '%%d' in the filename, but it actually doesn't. This has been fixed as well. > *) Input parameters such as width, height, sample_rate and ch_layout are now exposed at the level of the main() function and they are much easier to customize. > *) fprintf() has been replaced with av_log() > This list alone implies that this patch must be split into small, self-contained pieces that can be reviewed (whereas the current patch can't). There is just one thing that I immediately noticed: > > + AVCodecParameters audio_in_params = { > + .codec_type = AVMEDIA_TYPE_AUDIO, > + .format = AV_SAMPLE_FMT_S16, > + .sample_rate = 44100, > + .ch_layout = (AVChannelLayout)AV_CHANNEL_LAYOUT_STEREO > + }, > + video_in_params = { > + .codec_type = AVMEDIA_TYPE_VIDEO, > + .width = 352, > + .height = 288, > + .format = AV_PIX_FMT_YUV420P > + }, > + video_enc_params = { 0 }, audio_enc_params = { 0 }; sizeof(AVCodecParameters) is not public, you must not put AVCodecParameters on the stack. - 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".