From: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> To: ffmpeg-devel@ffmpeg.org Cc: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> Subject: [FFmpeg-devel] [PATCH 5/7] avutil: Move error.h from avutil.h to common.h Date: Sat, 2 Sep 2023 18:21:39 +0200 Message-ID: <AS8P250MB07448ABF701E1FCF1A0A84518FEBA@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> (raw) In-Reply-To: <AS8P250MB074451A1439E6A39CC721E838FEBA@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM> Up until now, avutil.h includes common.h which includes mem.h which includes avutil.h, so that all these headers are in fact equivalent. Yet mem.h does not need to include avutil.h at all and when it no longer does, including common.h will no longer include error.h (included by avutil.h) as well; change this by moving error.h to avutil.h, as error.h is clearly a commonly used header. Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com> --- libavutil/avutil.h | 1 - libavutil/common.h | 1 + 2 files changed, 1 insertion(+), 1 deletion(-) diff --git a/libavutil/avutil.h b/libavutil/avutil.h index 64b68bdbd3..30bec192f1 100644 --- a/libavutil/avutil.h +++ b/libavutil/avutil.h @@ -294,7 +294,6 @@ char av_get_picture_type_char(enum AVPictureType pict_type); */ #include "common.h" -#include "error.h" #include "rational.h" #include "version.h" #include "macros.h" diff --git a/libavutil/common.h b/libavutil/common.h index fd1404be6c..de2140a678 100644 --- a/libavutil/common.h +++ b/libavutil/common.h @@ -40,6 +40,7 @@ #include <string.h> #include "attributes.h" +#include "error.h" #include "macros.h" //rounded division & shift -- 2.34.1 _______________________________________________ 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".
next prev parent reply other threads:[~2023-09-02 16:24 UTC|newest] Thread overview: 20+ messages / expand[flat|nested] mbox.gz Atom feed top 2023-09-02 11:31 [FFmpeg-devel] [PATCH] avformat/avformat: Avoid including codec.h, frame.h Andreas Rheinhardt 2023-09-02 16:21 ` [FFmpeg-devel] [PATCH 2/7] avcodec/defs: Add AV_PROFILE_* defines, deprecate FF_PROFILE_* defines Andreas Rheinhardt 2023-09-02 16:21 ` [FFmpeg-devel] [PATCH 3/7] avformat/mux: Only write HEADER marker if format has .write_header Andreas Rheinhardt 2023-09-02 16:21 ` [FFmpeg-devel] [PATCH 4/7] avformat/aviobuf: Add ffio_init_(read|write)_context() Andreas Rheinhardt 2023-09-02 16:21 ` Andreas Rheinhardt [this message] 2023-09-03 7:47 ` [FFmpeg-devel] [PATCH 5/7] avutil: Move error.h from avutil.h to common.h Paul B Mahol 2023-09-02 16:21 ` [FFmpeg-devel] [PATCH 6/7] avutil/mem: Don't include avutil.h Andreas Rheinhardt 2023-09-03 7:43 ` Paul B Mahol 2023-09-04 14:05 ` Michael Niedermayer 2023-09-04 14:20 ` Andreas Rheinhardt 2023-09-02 16:21 ` [FFmpeg-devel] [PATCH 7/7] avutil/avstring: Remove obsolete version.h inclusion Andreas Rheinhardt 2023-09-02 23:12 ` [FFmpeg-devel] [PATCH 8/8] all: Replace __FUNCTION__ by __func__ Andreas Rheinhardt 2023-09-02 23:40 ` James Almer 2023-09-03 9:05 ` Andreas Rheinhardt 2023-09-02 23:28 ` [FFmpeg-devel] [PATCH 9/9] avcodec/vp8data: Use <> for inclusion of stdint.h Andreas Rheinhardt 2023-09-03 7:47 ` Paul B Mahol 2023-09-03 14:22 ` [FFmpeg-devel] [PATCH 10/11] avfilter/af_volume: Also deprecate position enum value Andreas Rheinhardt 2023-09-03 14:22 ` [FFmpeg-devel] [PATCH 11/11] avfilter/f_select: Also deprecate "pos" variable name Andreas Rheinhardt 2023-09-04 14:17 ` [FFmpeg-devel] [PATCH v2 6/11] avutil/mem: Don't include avutil.h Andreas Rheinhardt 2023-09-06 9:36 ` [FFmpeg-devel] [PATCH] avformat/avformat: Avoid including codec.h, frame.h 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=AS8P250MB07448ABF701E1FCF1A0A84518FEBA@AS8P250MB0744.EURP250.PROD.OUTLOOK.COM \ --to=andreas.rheinhardt@outlook.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