[FFmpeg-devel] PATCH] Make H.274 film grain support optional for H.264. Saves ~779kb.

Dale Curtis dalecurtis at chromium.org
Wed Aug 14 00:38:48 EEST 2024


On Tue, Aug 13, 2024 at 1:11 PM Hendrik Leppkes <h.leppkes at gmail.com> wrote:

> Disabling random codec features seems like an anti-feature to me, in
> the future it'll make every feature be questioned and compile-time
> conditional, and make everything terrible.
> If the context size is the major concern, maybe large structures
> should be allocated when in use, rather than always?
>

I agree with that, so here's a version which allocates dynamically instead.
It passes FATE, but I didn't try with valgrind/msan in case I missed a
cleanup path.

- dale
-------------- next part --------------
A non-text attachment was scrubbed...
Name: dynamic_film_grain_v0.patch
Type: application/octet-stream
Size: 11878 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240813/80597d34/attachment.obj>


More information about the ffmpeg-devel mailing list