[FFmpeg-devel] [PATCH 1/2] lavfi: regroup formats lists in a single structure.

[Nicolas George]

Andreas Rheinhardt (12020-08-21):
> This only works if one only works with one list without owner at a time;
> if one has several such lists at the same time and an error happens when
> working with one of these lists, one has to free the other lists.
> 
> This scenario can of course usually be avoided by making sure that one
> only has one list without owner at any given time. I used this method in
> my patches a lot. I have also rewritten the hwdownload patch [1] to
> avoid explicit ff_formats_unref() in it. This means that
> ff_formats_unref() can become static, too.
> 
> [1]: https://ffmpeg.org/pipermail/ffmpeg-devel/2020-August/268429.html

Well done.

> What does "this" refer to here? If it refers to my patches, then nothing
> is missing (except a review). They can be applied at once and then they
> can also be directly backported and you can then apply your big patch on
> top of it; of course, there will be trivial conflicts in the few files
> that I touch.
> 
> If "this" refers to your patches: Applying them before fixing the
> leaks/double-frees will make backporting the fixes much more troublesome.

I meant your patch series: I have become lost in the various
submissions. Are there still patches that need review?

Regards,

-- 
  Nicolas George
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 833 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20200822/e71f690b/attachment.sig>