Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
* [FFmpeg-devel] [PATCH] Don't reallocate a AVCodecContext when closing a non-open codec.
@ 2024-08-02 16:54 Dale Curtis
  2024-08-14 16:44 ` Dale Curtis
  0 siblings, 1 reply; 2+ messages in thread
From: Dale Curtis @ 2024-08-02 16:54 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

[-- Attachment #1: Type: text/plain, Size: 315 bytes --]

This results in an unnecessary ~800k allocation with H.264. A
nearby callsite uses avcodec_is_open() to avoid this, so do the
same when exiting avformat_find_stream_info().

Signed-off-by: Dale Curtis <dalecurtis@chromium.org>
---
 libavformat/demux.c | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

[-- Attachment #2: no_realloc_v1.patch --]
[-- Type: application/octet-stream, Size: 1153 bytes --]

[-- Attachment #3: Type: text/plain, Size: 251 bytes --]

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread

* Re: [FFmpeg-devel] [PATCH] Don't reallocate a AVCodecContext when closing a non-open codec.
  2024-08-02 16:54 [FFmpeg-devel] [PATCH] Don't reallocate a AVCodecContext when closing a non-open codec Dale Curtis
@ 2024-08-14 16:44 ` Dale Curtis
  0 siblings, 0 replies; 2+ messages in thread
From: Dale Curtis @ 2024-08-14 16:44 UTC (permalink / raw)
  To: FFmpeg development discussions and patches

Bump for this one. Thanks!

- dale

On Fri, Aug 2, 2024 at 9:54 AM Dale Curtis <dalecurtis@chromium.org> wrote:

> This results in an unnecessary ~800k allocation with H.264. A
> nearby callsite uses avcodec_is_open() to avoid this, so do the
> same when exiting avformat_find_stream_info().
>
> Signed-off-by: Dale Curtis <dalecurtis@chromium.org>
> ---
>  libavformat/demux.c | 9 ++++++---
>  1 file changed, 6 insertions(+), 3 deletions(-)
>
>
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

^ permalink raw reply	[flat|nested] 2+ messages in thread

end of thread, other threads:[~2024-08-14 16:44 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-08-02 16:54 [FFmpeg-devel] [PATCH] Don't reallocate a AVCodecContext when closing a non-open codec Dale Curtis
2024-08-14 16:44 ` Dale Curtis

Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
		ffmpegdev@gitmailbox.com
	public-inbox-index ffmpegdev

Example config snippet for mirrors.


AGPL code for this site: git clone https://public-inbox.org/public-inbox.git