[FFmpeg-devel] [PATCH 2/2] doc/encoders: very basic FFv1 documentation

[Michael Niedermayer]

Hi

On Tue, Nov 26, 2024 at 11:48:54AM +0100, Anton Khirnov wrote:
> Quoting Michael Niedermayer (2024-10-15 00:46:50)
> > + at item qtable
> > +-1 (default, automatic), 0 use 8bit default, 1, use >8bit default
> 
> int seems like the wrong type then.

well, you are correct, (maybe not they way you meant it)
but as the encoder can choose any table of any values.
an int is not able to convey that and something like a string would
be better. Allowing explicit values as well as automcatic and specific
named tables

Currently its an int though. And an enum may be undefined
tha AVOption should maybe also have some named constants instead
of 0,1

before i implement any of this, which direction was your comment
meant to point to ?

thx

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

In fact, the RIAA has been known to suggest that students drop out
of college or go to community college in order to be able to afford
settlements. -- The RIAA
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 195 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20241127/7f81389f/attachment.sig>