[FFmpeg-devel] [PATCH] lavu/tx: stop using av_log(NULL, )
Lynne
dev at lynne.ee
Fri Jul 26 11:33:24 EEST 2024
On 26/07/2024 09:47, Anton Khirnov wrote:
> Quoting Lynne via ffmpeg-devel (2024-07-26 08:42:11)
>> Its not feasible to add an AVClass in the main context, as
>> it would waste space, as the main context is recursive, and
>> every bit of assembly would need to be changed.
>>
>> While its true that on paper av_log has access to the main
>> context, that functionality is not used as no options are
>> available for setting. No options will be exposed either,
>> and it makes no sense.
>>
>> mpv has recently started warning if a NULL AVClass is used
>> as an FFmpeg bug. While I don't fully agree nor disagree with
>> this, this is a simple patch which fixes the issue.
>
> No, it just hides the issue for the time being.
If this means "it may get broken eventually, its not forbidden
anywhere", then IMO we should just codify the current behavior such that
it won't, unless there's some use-case you can think of.
> I am against this patch, just add a proper AVClass. AVTXContext is
> entirely opaque, so it should definitely be feasible.
I'd like to avoid adding a pointer and allocating it if it can't be
helped. And properly integrating each context into the AVClass system as
a child of the parent context.
If you think a NULL av_log is valid (you implied that a year ago), then
I'm more than happy to drop this patch.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_0xA2FEA5F03F034464.asc
Type: application/pgp-keys
Size: 624 bytes
Desc: OpenPGP public key
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240726/a8ffab10/attachment.key>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: OpenPGP_signature.asc
Type: application/pgp-signature
Size: 236 bytes
Desc: OpenPGP digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20240726/a8ffab10/attachment.sig>
More information about the ffmpeg-devel
mailing list