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 424A942915 for ; Tue, 7 Jun 2022 09:42:53 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id 5BE8E68B5ED; Tue, 7 Jun 2022 12:42:51 +0300 (EEST) Received: from mail0.khirnov.net (red.khirnov.net [176.97.15.12]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 7682268B2A8 for ; Tue, 7 Jun 2022 12:42:44 +0300 (EEST) Received: from localhost (localhost [IPv6:::1]) by mail0.khirnov.net (Postfix) with ESMTP id B02D8240175 for ; Tue, 7 Jun 2022 11:42:43 +0200 (CEST) Received: from mail0.khirnov.net ([IPv6:::1]) by localhost (mail0.khirnov.net [IPv6:::1]) (amavisd-new, port 10024) with ESMTP id qrT2xQfGkTuz for ; Tue, 7 Jun 2022 11:42:43 +0200 (CEST) Received: from lain.khirnov.net (lain.khirnov.net [IPv6:2001:67c:1138:4306::3]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits) server-digest SHA256 client-signature RSA-PSS (2048 bits) client-digest SHA256) (Client CN "lain.khirnov.net", Issuer "smtp.khirnov.net SMTP CA" (verified OK)) by mail0.khirnov.net (Postfix) with ESMTPS id 27BAE2400F5 for ; Tue, 7 Jun 2022 11:42:43 +0200 (CEST) Received: by lain.khirnov.net (Postfix, from userid 1000) id 3D6A41601B2; Tue, 7 Jun 2022 11:42:43 +0200 (CEST) From: Anton Khirnov To: FFmpeg Development Discussions and Patches In-Reply-To: <580594123.12475702.1654537242853@mail.yahoo.com> References: <580594123.12475702.1654537242853.ref@mail.yahoo.com> <580594123.12475702.1654537242853@mail.yahoo.com> Mail-Followup-To: FFmpeg Development Discussions and Patches Date: Tue, 07 Jun 2022 11:42:43 +0200 Message-ID: <165459496322.13099.10162954010675311855@lain> User-Agent: alot/0.8.1 MIME-Version: 1.0 Subject: Re: [FFmpeg-devel] [PATCH] New API usage example (reading, converting, encoding and muxing an audio file) 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: Quoting Paolo Prete (2022-06-06 19:40:42) > Hello, > From what I see, the doc/examples files can be still improved. More > precisely, their main() is often split into functions with short names > that hide important (sometime essential) details of what they're doing > and they worsen the readability, by forcing the user to jump from > chunks to chunks of the code. I disagree VERY strongly. Long functions are extremely hard to reason about, because of all the pieces you have to keep in your head at once. Grouping logically related blocks of code into their own small functions that can be thought of as a single unit immensely improves readability and maintainability. -- Anton Khirnov _______________________________________________ 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".