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 D2EBC48CE6 for ; Tue, 23 Apr 2024 20:24:06 +0000 (UTC) Received: from [127.0.1.1] (localhost [127.0.0.1]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTP id C24B668D385; Tue, 23 Apr 2024 23:24:03 +0300 (EEST) Received: from mail-pl1-f174.google.com (mail-pl1-f174.google.com [209.85.214.174]) by ffbox0-bg.mplayerhq.hu (Postfix) with ESMTPS id 1CA2C68D28C for ; Tue, 23 Apr 2024 23:23:58 +0300 (EEST) Received: by mail-pl1-f174.google.com with SMTP id d9443c01a7336-1e9451d8b71so27944875ad.0 for ; Tue, 23 Apr 2024 13:23:57 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20230601; t=1713903835; x=1714508635; darn=ffmpeg.org; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id:from :to:cc:subject:date:message-id:reply-to; bh=7YpIskhRTps3owVRuiTJrUTd2YYs4rCA2PoXEkE2p5M=; b=NAMys97E7kWQJH/D/GyeDOOTrxh8RLHPwQDxpF0yWm+br7ljt7e4FYhfc7Zj/QZvHZ dfurmh3FG0wxpe4+slU+i7NZ7/cwe5uLVy6WDN8MFFct55C21VOxUFWjz9XqO9xDOdj9 NYLIcEtOL3C5rglfps4EjU3uMeAyOeW5l200tWWu1SAnGZFl8LE5BArxHeGwSbuPK9ug O13tAur/4rGPfVlsmaWnfMC0YWK3g5KYQYf6klV/EE5TepEOAwrikIPoj4DG70VbRZ5r +xuBMK5kaCgjpLfLotDv3ZFn4WejXKfNMTJ28uOoz+zFRWCJHBHgWPmbzzXGB56jXWty rKPw== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20230601; t=1713903835; x=1714508635; h=content-transfer-encoding:in-reply-to:from:content-language :references:to:subject:user-agent:mime-version:date:message-id :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to; bh=7YpIskhRTps3owVRuiTJrUTd2YYs4rCA2PoXEkE2p5M=; b=YFq5dkvdw9zlNh62U1LAH4kEcfjwkd3n4uXQq2Bw9rjB3M/oEzWYBnRQ/T6E8o/dYG mN23seWCRiWzWBO0Xrz0r6KS4F96Q9mXpvkEv74vkUAPWEUDki1i+y7Tx/+zYEIp+5sp jmaiuauo3d6ICva40doLaFHtY12Dl54d50L0PtqYsV4H/oYoOzFLDLmNCYzMIVuLjnmI NA8El22SEVMvV9d+6VjqmBHaAudGHidzZKSF8VTnHokMGMlHc569cfkbaXUcMK/SwtEX quYlbmvTDsoFcxZseAtduHEpNoQrNdAVLylUHw09me6a98Em8cgsaaKCyr5+mRmXcFZH +uWA== X-Gm-Message-State: AOJu0YzpXC2aMkVjtKb+1m1m7W3Dlw4PoLv7bDbqzKKghHR4aoQaWZz/ s6auAUvE33xMaFSOKJhTDVS2sicLVzRucNT6NClmSsPB5GU5tC+bSRvZFw== X-Google-Smtp-Source: AGHT+IHzeP/jzS2cmi6K+23pJ+UEDH0r5AGVW4+BkkQUm4oSlvVS126HrWe0vxhVz/rdcC9vDzpnsw== X-Received: by 2002:a17:902:d2c1:b0:1e7:e7ed:7787 with SMTP id n1-20020a170902d2c100b001e7e7ed7787mr580768plc.51.1713903834779; Tue, 23 Apr 2024 13:23:54 -0700 (PDT) Received: from [192.168.0.10] ([190.194.167.233]) by smtp.gmail.com with ESMTPSA id u2-20020a170902e80200b001e43e111baasm10436365plg.309.2024.04.23.13.23.53 for (version=TLS1_3 cipher=TLS_AES_128_GCM_SHA256 bits=128/128); Tue, 23 Apr 2024 13:23:54 -0700 (PDT) Message-ID: <927b5c94-c765-4852-a502-4369b4a624c6@gmail.com> Date: Tue, 23 Apr 2024 17:24:03 -0300 MIME-Version: 1.0 User-Agent: Mozilla Thunderbird To: ffmpeg-devel@ffmpeg.org References: <20240422120925.380253-1-ffmpeg-devel@pileofstuff.org> <171386408794.22242.4320691091356910947@lain.khirnov.net> <171386667402.5069.14623062002271611181@lain.khirnov.net> <20240423111552.GI6420@pb2> Content-Language: en-US From: James Almer In-Reply-To: <20240423111552.GI6420@pb2> Subject: Re: [FFmpeg-devel] [PATCH v2] lavu/opt: Clarify that AVOptions is not indended for general use 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-Transfer-Encoding: 7bit Content-Type: text/plain; charset="us-ascii"; Format="flowed" Errors-To: ffmpeg-devel-bounces@ffmpeg.org Sender: "ffmpeg-devel" Archived-At: List-Archive: List-Post: On 4/23/2024 8:15 AM, Michael Niedermayer wrote: > On Tue, Apr 23, 2024 at 11:10:43AM +0100, Andrew Sayers wrote: >> On Tue, Apr 23, 2024 at 12:04:34PM +0200, Anton Khirnov wrote: >>> Quoting Andrew Sayers (2024-04-23 11:51:00) >>>> On Tue, Apr 23, 2024 at 11:21:27AM +0200, Anton Khirnov wrote: >>>>>> lavu/opt: Clarify that AVOptions is not indended for general use >>>>> >>>>> They _are_ intended for general use though. >>>> >>>> In that case I'm confused... >>>> >>>> Let's say I make a desktop app to transcode videos. Obviously I would use >>>> AVOptions to display configuration options for different encoders. And it's >>>> possible to create AVOptions objects for my UI. But how strongly is that use >>>> case recommended? >>>> >>>> To provide a particularly difficult example - let's say I want to let the user >>>> choose between interface themes, and I want to show both some text and a >>>> picture of the theme. AVOption doesn't include a "text + picture" option, >>>> so how would I extend it to meet my needs? >>> >>> If they fit your use case, then use them, otherwise don't - that's true >>> for pretty much all APIs we provide. >> >> Ah ok, so how about if I changed "intended" to "optimized" in the subject? > > If FFmpeg which is a multimedia tool in no place needs or wants to store > pictures through its option API in a way not curently supported. > I would say thats not going to qualify as "general use" outside specialized > software thats already dealing with a lot of pictures > > still you certainly can handle binary data (like a bitmap picture) through > AVOption > > thx Take for example AVIAMFReconGain.recon_gain in libavutil/iamf.h, which is currently the only field not covered by an AVOption (And thus not currently configurable from the CLI). How could it be supported? Binary type doesn't work because it expects a pointer + size field and allocates the former. _______________________________________________ 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".