Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
 help / color / mirror / Atom feed
From: Anton Khirnov <anton@khirnov.net>
To: FFmpeg development discussions and patches <ffmpeg-devel@ffmpeg.org>
Cc: Marvin Scholz <epirat07@gmail.com>
Subject: Re: [FFmpeg-devel] [PATCH 3/3] avutil/dict: constify av_dict_get return
Date: Mon, 05 Jun 2023 12:05:32 +0200
Message-ID: <168595953224.3843.11383068687026739305@lain.khirnov.net> (raw)
In-Reply-To: <20230501114456.13898-3-epirat07@gmail.com>

Quoting Marvin Scholz (2023-05-01 13:44:56)
> ---
>  doc/examples/qsv_transcode.c       |  2 +-
>  libavcodec/libaomenc.c             |  2 +-
>  libavcodec/libkvazaar.c            |  2 +-
>  libavcodec/libsvtav1.c             |  2 +-
>  libavcodec/libvpxenc.c             |  2 +-
>  libavcodec/libx264.c               |  2 +-
>  libavcodec/libx265.c               |  2 +-
>  libavcodec/mjpegdec.c              |  2 +-
>  libavcodec/qsvenc.c                |  2 +-
>  libavfilter/avfilter.c             |  2 +-
>  libavfilter/f_bench.c              |  2 +-
>  libavfilter/f_drawgraph.c          |  2 +-
>  libavfilter/f_select.c             |  4 +--
>  libavfilter/vf_cover_rect.c        |  2 +-
>  libavfilter/vf_drawtext.c          |  2 +-
>  libavformat/aiffenc.c              |  2 +-
>  libavformat/argo_asf.c             |  2 +-
>  libavformat/asfenc.c               |  6 ++--
>  libavformat/au.c                   |  2 +-
>  libavformat/avformat.h             |  2 +-
>  libavformat/avidec.c               |  2 +-
>  libavformat/avienc.c               |  2 +-
>  libavformat/avio.c                 |  4 +--
>  libavformat/dashenc.c              |  6 ++--
>  libavformat/dvenc.c                |  2 +-
>  libavformat/flacdec.c              |  2 +-
>  libavformat/flacenc.c              |  6 ++--
>  libavformat/flvdec.c               |  2 +-
>  libavformat/gxfenc.c               |  2 +-
>  libavformat/http.c                 |  8 +++---
>  libavformat/id3v2.c                |  6 ++--
>  libavformat/id3v2enc.c             |  2 +-
>  libavformat/imfdec.c               |  2 +-
>  libavformat/matroskadec.c          |  2 +-
>  libavformat/mov.c                  |  6 ++--
>  libavformat/movenc.c               | 46 +++++++++++++++---------------
>  libavformat/mp3enc.c               |  6 ++--
>  libavformat/mpegtsenc.c            |  6 ++--
>  libavformat/mux.c                  |  2 +-
>  libavformat/mux_utils.c            |  2 +-
>  libavformat/mxfenc.c               | 14 ++++-----
>  libavformat/riffenc.c              |  2 +-
>  libavformat/rmenc.c                |  2 +-
>  libavformat/sapenc.c               |  2 +-
>  libavformat/sdp.c                  |  2 +-
>  libavformat/segment.c              |  4 +--
>  libavformat/soxenc.c               |  2 +-
>  libavformat/tee.c                  |  2 +-
>  libavformat/ttmlenc.c              |  4 +--
>  libavformat/wavenc.c               |  4 +--
>  libavformat/webmdashenc.c          | 30 +++++++++----------
>  libavutil/dict.c                   |  9 +++++-
>  libavutil/dict.h                   |  5 ++++
>  libavutil/hwcontext_cuda.c         |  2 +-
>  libavutil/hwcontext_qsv.c          |  2 +-
>  libavutil/hwcontext_vulkan.c       |  8 +++---
>  libavutil/version.h                |  1 +
>  tests/api/api-threadmessage-test.c |  2 +-
>  58 files changed, 136 insertions(+), 123 deletions(-)

IMO this is an API break, and so should be postponed until the next
major bump.

-- 
Anton Khirnov
_______________________________________________
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".

  reply	other threads:[~2023-06-05 10:05 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2023-05-01 11:44 [FFmpeg-devel] [PATCH 1/3] avutil/dict: add av_dict_pop Marvin Scholz
2023-05-01 11:44 ` [FFmpeg-devel] [PATCH 2/3] avformat/tee: use av_dict_pop Marvin Scholz
2023-06-25 12:07   ` "zhilizhao(赵志立)"
2023-05-01 11:44 ` [FFmpeg-devel] [PATCH 3/3] avutil/dict: constify av_dict_get return Marvin Scholz
2023-06-05 10:05   ` Anton Khirnov [this message]
2023-05-21 23:52 ` [FFmpeg-devel] [PATCH 1/3] avutil/dict: add av_dict_pop Stefano Sabatini
2023-05-22  9:23   ` Marvin Scholz
2023-05-26  6:05     ` Stefano Sabatini
2023-05-26  9:11       ` Marvin Scholz
2023-05-26 20:02         ` Michael Niedermayer
2023-05-26 20:51           ` Marvin Scholz
2023-05-26 20:06         ` James Almer
2023-06-04 14:25         ` Stefano Sabatini
2023-06-04 14:34           ` Marvin Scholz
2023-06-05  8:08             ` Stefano Sabatini
2023-06-05 10:09               ` Anton Khirnov
2023-06-05 10:04 ` Anton Khirnov
2023-06-25 10:49 ` [FFmpeg-devel] [PATCH v2 " Marvin Scholz
2023-06-25 10:49   ` [FFmpeg-devel] [PATCH v2 2/3] avformat/tee: use av_dict_pop Marvin Scholz
2023-07-02  8:47     ` Stefano Sabatini
2023-06-25 10:49   ` [FFmpeg-devel] [PATCH v2 3/3] avutil/dict: constify av_dict_get return Marvin Scholz
2023-07-02  9:06     ` Stefano Sabatini
2023-07-02  8:43   ` [FFmpeg-devel] [PATCH v2 1/3] avutil/dict: add av_dict_pop Stefano Sabatini
2023-07-02 11:49     ` Marvin Scholz
2023-07-03  0:18   ` Andreas Rheinhardt
2023-07-03  9:11     ` Marvin Scholz
2023-07-03 18:02       ` Andreas Rheinhardt
2023-07-03 22:41         ` Marvin Scholz
2023-10-20  8:18         ` Anton Khirnov
2023-10-20 14:00           ` Andreas Rheinhardt
2023-10-20 14:33             ` Anton Khirnov
2023-10-20 15:42               ` Andreas Rheinhardt

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=168595953224.3843.11383068687026739305@lain.khirnov.net \
    --to=anton@khirnov.net \
    --cc=epirat07@gmail.com \
    --cc=ffmpeg-devel@ffmpeg.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link

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