[FFmpeg-devel] Replies on the mailing-list (was: [PATCH] WIP: avutil/pixdesc: reduce the size of symbols in pixdesc to optimize code size. (PR #20154))

[Nicolas George]

Nicolas George (HE12025-08-07):
> > URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20154
> > Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20154.patch

> I do not like that it bloats the public headers and creates a
> binary-incompatible version of the library.
> 
> And IIRC you cannot use CONFIG_SOMETHING in public headers anyway.
> 
> Better leave the function as stubs in the libraries.

My comment did not reach the web monster, even though I overrode the
reply-to header to include code at ffmpeg.org in the reply.

Still a lot of details to fix to get this thing anywhere remotely
usable.

-- 
  Nicolas George