[FFmpeg-devel] [PATCH 2/2] lavfi: make filter_frame non-recursive.
george at nsup.org
Mon Nov 14 01:18:12 EET 2016
Le quartidi 24 brumaire, an CCXXV, Hendrik Leppkes a écrit :
> I disagree with that, this is generic utility code, not a tight
> algorithm loop, where absolute efficiency is not necessarily required,
> and disregarding code simplicity amd consistency will just harm in the
We can discuss that. Remember that they are fields in a structure, not
code: if these fields are accessed in a tight loop, your argument
I agree that most of these fields will likely not be used in very tight
loops. But "most", "likely", "very", because of these caveats, the
concern can not be dismissed entirely.
And remember that the indirection also harms the code simplicity,
I claim that my solution yields overall simpler code. And more
(I could play and say that if you do not see how the indirection harms
the simplicity of the code, your advice is worthless on the matter, but
that would be vile. If you do not see, ask and I will explain; just not
right now because I need to wake up early tomorrow.)
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 801 bytes
Desc: Digital signature
More information about the ffmpeg-devel