[FFmpeg-devel] [PATCH 1/2] avformat/tee: refactor option processing

Nicolas George george at nsup.org
Thu Jul 3 12:08:00 EEST 2025


Marvin Scholz (HE12025-06-25):
> Would you be fine with just the removal of the messing with
> the AVDictionary entries then, leaving the macros in place,
> essentially removing STEAL_OPTION and doing a copy in CONSUME_OPTION?
> 
> IMHO saving two copies of a string does not justify abusing the
> AVDictionary API in such a way. This isnt a hot code path either
> where this would make sense...

It is not just a matter of saving a few cycles. What you propose
requires writing more code, including error checks and an occasion for
failure.

If it was new code, I would consider it, but changing existing code that
has been working for years to make it more verbose, more failure prone
and less efficient, no, thanks.

Let us just document that it is a valid use.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list