[FFmpeg-devel] [PATCHv2 1/3] avutil/eval: separate AVExpr state to a new AVExprState struct

Anton Khirnov anton at khirnov.net
Fri Jan 10 12:32:44 EET 2020


Quoting Marton Balint (2019-12-30 23:23:40)
> Also add helper functions to allocate and free such a struct, and make it
> usable by providing a new av_eval_expr2 function for which you can specify a
> custom AVExprState.

Why not just parse the expression multiple times? Performance?

Beyond that, it seems to me that the user now has to juggle multiple
contexts manually, which is complex and can be avoided. How about
introducing a function for duplicating AVExpr instead? Would that not
solve this as well?

-- 
Anton Khirnov


More information about the ffmpeg-devel mailing list