[FFmpeg-devel] [PATCH] Use NULL_IF_CONFIG_SMALL for AVOption tables.

Marton Balint cus at passwd.hu
Mon Aug 28 23:52:44 EEST 2017


On Mon, 28 Aug 2017, Dale Curtis wrote:

> On Mon, Aug 28, 2017 at 2:28 AM, wm4 <nfxjfg at googlemail.com> wrote:
>>
>>
>> Seems way too intrusive and the result makes AVOption tables even
>> uglier and harder to read.
>
>
> Agree it's ugly. Any suggestions? I tried some way of forcing the AVOption
> entry to have zero-storage (thus null always), but my language-fu was not
> strong enough. Moving options data into some sort of data file and
> programmatically generating the tables might work.

Have you considered using/creating a semantic patch to change these? I bet 
there are other semantically well-defined cases where you can get rid of 
text if you are really aiming for reduced size.

I'd love to see that kind of text reducing spatch maintained for people who 
really aiming for small size, and with it probably we can get rid of most 
of the existing NULL_IF_CONFIG_SMALL cases as well.

Regards,
Marton


More information about the ffmpeg-devel mailing list