* [FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header
@ 2024-03-07 15:04 Andreas Rheinhardt
2024-03-07 15:21 ` James Almer
0 siblings, 1 reply; 2+ messages in thread
From: Andreas Rheinhardt @ 2024-03-07 15:04 UTC (permalink / raw)
To: ffmpeg-devel; +Cc: Andreas Rheinhardt
Forgotten in a12cd3be98e8aba6e74274192ec6532988aa9444.
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
libavcodec/Makefile | 2 -
libavcodec/xvmc.h | 171 --------------------------------------------
2 files changed, 173 deletions(-)
delete mode 100644 libavcodec/xvmc.h
diff --git a/libavcodec/Makefile b/libavcodec/Makefile
index 93f6cacf00..5d99120aa9 100644
--- a/libavcodec/Makefile
+++ b/libavcodec/Makefile
@@ -25,7 +25,6 @@ HEADERS = ac3_parser.h \
version_major.h \
videotoolbox.h \
vorbis_parser.h \
- xvmc.h \
OBJS = ac3_parser.o \
adts_parser.o \
@@ -1278,7 +1277,6 @@ SKIPHEADERS-$(CONFIG_NVDEC) += nvdec.h
SKIPHEADERS-$(CONFIG_NVENC) += nvenc.h
SKIPHEADERS-$(CONFIG_QSV) += qsv.h qsv_internal.h
SKIPHEADERS-$(CONFIG_QSVENC) += qsvenc.h
-SKIPHEADERS-$(CONFIG_XVMC) += xvmc.h
SKIPHEADERS-$(CONFIG_VAAPI) += vaapi_decode.h vaapi_hevc.h vaapi_encode.h
SKIPHEADERS-$(CONFIG_VDPAU) += vdpau.h vdpau_internal.h
SKIPHEADERS-$(CONFIG_VIDEOTOOLBOX) += videotoolbox.h vt_internal.h
diff --git a/libavcodec/xvmc.h b/libavcodec/xvmc.h
deleted file mode 100644
index 52e70c0d77..0000000000
--- a/libavcodec/xvmc.h
+++ /dev/null
@@ -1,171 +0,0 @@
-/*
- * Copyright (C) 2003 Ivan Kalvachev
- *
- * This file is part of FFmpeg.
- *
- * FFmpeg is free software; you can redistribute it and/or
- * modify it under the terms of the GNU Lesser General Public
- * License as published by the Free Software Foundation; either
- * version 2.1 of the License, or (at your option) any later version.
- *
- * FFmpeg is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
- * Lesser General Public License for more details.
- *
- * You should have received a copy of the GNU Lesser General Public
- * License along with FFmpeg; if not, write to the Free Software
- * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
- */
-
-#ifndef AVCODEC_XVMC_H
-#define AVCODEC_XVMC_H
-
-/**
- * @file
- * @ingroup lavc_codec_hwaccel_xvmc
- * Public libavcodec XvMC header.
- */
-
-#pragma message("XvMC is no longer supported; this header is deprecated and will be removed")
-
-#include <X11/extensions/XvMC.h>
-
-#include "libavutil/attributes.h"
-#include "avcodec.h"
-
-/**
- * @defgroup lavc_codec_hwaccel_xvmc XvMC
- * @ingroup lavc_codec_hwaccel
- *
- * @{
- */
-
-#define AV_XVMC_ID 0x1DC711C0 /**< special value to ensure that regular pixel routines haven't corrupted the struct
- the number is 1337 speak for the letters IDCT MCo (motion compensation) */
-
-struct attribute_deprecated xvmc_pix_fmt {
- /** The field contains the special constant value AV_XVMC_ID.
- It is used as a test that the application correctly uses the API,
- and that there is no corruption caused by pixel routines.
- - application - set during initialization
- - libavcodec - unchanged
- */
- int xvmc_id;
-
- /** Pointer to the block array allocated by XvMCCreateBlocks().
- The array has to be freed by XvMCDestroyBlocks().
- Each group of 64 values represents one data block of differential
- pixel information (in MoCo mode) or coefficients for IDCT.
- - application - set the pointer during initialization
- - libavcodec - fills coefficients/pixel data into the array
- */
- short* data_blocks;
-
- /** Pointer to the macroblock description array allocated by
- XvMCCreateMacroBlocks() and freed by XvMCDestroyMacroBlocks().
- - application - set the pointer during initialization
- - libavcodec - fills description data into the array
- */
- XvMCMacroBlock* mv_blocks;
-
- /** Number of macroblock descriptions that can be stored in the mv_blocks
- array.
- - application - set during initialization
- - libavcodec - unchanged
- */
- int allocated_mv_blocks;
-
- /** Number of blocks that can be stored at once in the data_blocks array.
- - application - set during initialization
- - libavcodec - unchanged
- */
- int allocated_data_blocks;
-
- /** Indicate that the hardware would interpret data_blocks as IDCT
- coefficients and perform IDCT on them.
- - application - set during initialization
- - libavcodec - unchanged
- */
- int idct;
-
- /** In MoCo mode it indicates that intra macroblocks are assumed to be in
- unsigned format; same as the XVMC_INTRA_UNSIGNED flag.
- - application - set during initialization
- - libavcodec - unchanged
- */
- int unsigned_intra;
-
- /** Pointer to the surface allocated by XvMCCreateSurface().
- It has to be freed by XvMCDestroySurface() on application exit.
- It identifies the frame and its state on the video hardware.
- - application - set during initialization
- - libavcodec - unchanged
- */
- XvMCSurface* p_surface;
-
-/** Set by the decoder before calling ff_draw_horiz_band(),
- needed by the XvMCRenderSurface function. */
-//@{
- /** Pointer to the surface used as past reference
- - application - unchanged
- - libavcodec - set
- */
- XvMCSurface* p_past_surface;
-
- /** Pointer to the surface used as future reference
- - application - unchanged
- - libavcodec - set
- */
- XvMCSurface* p_future_surface;
-
- /** top/bottom field or frame
- - application - unchanged
- - libavcodec - set
- */
- unsigned int picture_structure;
-
- /** XVMC_SECOND_FIELD - 1st or 2nd field in the sequence
- - application - unchanged
- - libavcodec - set
- */
- unsigned int flags;
-//}@
-
- /** Number of macroblock descriptions in the mv_blocks array
- that have already been passed to the hardware.
- - application - zeroes it on get_buffer().
- A successful ff_draw_horiz_band() may increment it
- with filled_mb_block_num or zero both.
- - libavcodec - unchanged
- */
- int start_mv_blocks_num;
-
- /** Number of new macroblock descriptions in the mv_blocks array (after
- start_mv_blocks_num) that are filled by libavcodec and have to be
- passed to the hardware.
- - application - zeroes it on get_buffer() or after successful
- ff_draw_horiz_band().
- - libavcodec - increment with one of each stored MB
- */
- int filled_mv_blocks_num;
-
- /** Number of the next free data block; one data block consists of
- 64 short values in the data_blocks array.
- All blocks before this one have already been claimed by placing their
- position into the corresponding block description structure field,
- that are part of the mv_blocks array.
- - application - zeroes it on get_buffer().
- A successful ff_draw_horiz_band() may zero it together
- with start_mb_blocks_num.
- - libavcodec - each decoded macroblock increases it by the number
- of coded blocks it contains.
- */
- int next_free_data_block_num;
-};
-
-/**
- * @}
- */
-
-#endif /* AVCODEC_XVMC_H */
--
2.40.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".
^ permalink raw reply [flat|nested] 2+ messages in thread
* Re: [FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header
2024-03-07 15:04 [FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header Andreas Rheinhardt
@ 2024-03-07 15:21 ` James Almer
0 siblings, 0 replies; 2+ messages in thread
From: James Almer @ 2024-03-07 15:21 UTC (permalink / raw)
To: ffmpeg-devel
On 3/7/2024 12:04 PM, Andreas Rheinhardt wrote:
> Forgotten in a12cd3be98e8aba6e74274192ec6532988aa9444.
>
> Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
> ---
> libavcodec/Makefile | 2 -
> libavcodec/xvmc.h | 171 --------------------------------------------
> 2 files changed, 173 deletions(-)
> delete mode 100644 libavcodec/xvmc.h
>
> diff --git a/libavcodec/Makefile b/libavcodec/Makefile
> index 93f6cacf00..5d99120aa9 100644
> --- a/libavcodec/Makefile
> +++ b/libavcodec/Makefile
> @@ -25,7 +25,6 @@ HEADERS = ac3_parser.h \
> version_major.h \
> videotoolbox.h \
> vorbis_parser.h \
> - xvmc.h \
>
> OBJS = ac3_parser.o \
> adts_parser.o \
> @@ -1278,7 +1277,6 @@ SKIPHEADERS-$(CONFIG_NVDEC) += nvdec.h
> SKIPHEADERS-$(CONFIG_NVENC) += nvenc.h
> SKIPHEADERS-$(CONFIG_QSV) += qsv.h qsv_internal.h
> SKIPHEADERS-$(CONFIG_QSVENC) += qsvenc.h
> -SKIPHEADERS-$(CONFIG_XVMC) += xvmc.h
> SKIPHEADERS-$(CONFIG_VAAPI) += vaapi_decode.h vaapi_hevc.h vaapi_encode.h
> SKIPHEADERS-$(CONFIG_VDPAU) += vdpau.h vdpau_internal.h
> SKIPHEADERS-$(CONFIG_VIDEOTOOLBOX) += videotoolbox.h vt_internal.h
> diff --git a/libavcodec/xvmc.h b/libavcodec/xvmc.h
> deleted file mode 100644
> index 52e70c0d77..0000000000
> --- a/libavcodec/xvmc.h
> +++ /dev/null
> @@ -1,171 +0,0 @@
> -/*
> - * Copyright (C) 2003 Ivan Kalvachev
> - *
> - * This file is part of FFmpeg.
> - *
> - * FFmpeg is free software; you can redistribute it and/or
> - * modify it under the terms of the GNU Lesser General Public
> - * License as published by the Free Software Foundation; either
> - * version 2.1 of the License, or (at your option) any later version.
> - *
> - * FFmpeg is distributed in the hope that it will be useful,
> - * but WITHOUT ANY WARRANTY; without even the implied warranty of
> - * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the GNU
> - * Lesser General Public License for more details.
> - *
> - * You should have received a copy of the GNU Lesser General Public
> - * License along with FFmpeg; if not, write to the Free Software
> - * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
> - */
> -
> -#ifndef AVCODEC_XVMC_H
> -#define AVCODEC_XVMC_H
> -
> -/**
> - * @file
> - * @ingroup lavc_codec_hwaccel_xvmc
> - * Public libavcodec XvMC header.
> - */
> -
> -#pragma message("XvMC is no longer supported; this header is deprecated and will be removed")
> -
> -#include <X11/extensions/XvMC.h>
> -
> -#include "libavutil/attributes.h"
> -#include "avcodec.h"
> -
> -/**
> - * @defgroup lavc_codec_hwaccel_xvmc XvMC
> - * @ingroup lavc_codec_hwaccel
> - *
> - * @{
> - */
> -
> -#define AV_XVMC_ID 0x1DC711C0 /**< special value to ensure that regular pixel routines haven't corrupted the struct
> - the number is 1337 speak for the letters IDCT MCo (motion compensation) */
> -
> -struct attribute_deprecated xvmc_pix_fmt {
> - /** The field contains the special constant value AV_XVMC_ID.
> - It is used as a test that the application correctly uses the API,
> - and that there is no corruption caused by pixel routines.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - int xvmc_id;
> -
> - /** Pointer to the block array allocated by XvMCCreateBlocks().
> - The array has to be freed by XvMCDestroyBlocks().
> - Each group of 64 values represents one data block of differential
> - pixel information (in MoCo mode) or coefficients for IDCT.
> - - application - set the pointer during initialization
> - - libavcodec - fills coefficients/pixel data into the array
> - */
> - short* data_blocks;
> -
> - /** Pointer to the macroblock description array allocated by
> - XvMCCreateMacroBlocks() and freed by XvMCDestroyMacroBlocks().
> - - application - set the pointer during initialization
> - - libavcodec - fills description data into the array
> - */
> - XvMCMacroBlock* mv_blocks;
> -
> - /** Number of macroblock descriptions that can be stored in the mv_blocks
> - array.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - int allocated_mv_blocks;
> -
> - /** Number of blocks that can be stored at once in the data_blocks array.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - int allocated_data_blocks;
> -
> - /** Indicate that the hardware would interpret data_blocks as IDCT
> - coefficients and perform IDCT on them.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - int idct;
> -
> - /** In MoCo mode it indicates that intra macroblocks are assumed to be in
> - unsigned format; same as the XVMC_INTRA_UNSIGNED flag.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - int unsigned_intra;
> -
> - /** Pointer to the surface allocated by XvMCCreateSurface().
> - It has to be freed by XvMCDestroySurface() on application exit.
> - It identifies the frame and its state on the video hardware.
> - - application - set during initialization
> - - libavcodec - unchanged
> - */
> - XvMCSurface* p_surface;
> -
> -/** Set by the decoder before calling ff_draw_horiz_band(),
> - needed by the XvMCRenderSurface function. */
> -//@{
> - /** Pointer to the surface used as past reference
> - - application - unchanged
> - - libavcodec - set
> - */
> - XvMCSurface* p_past_surface;
> -
> - /** Pointer to the surface used as future reference
> - - application - unchanged
> - - libavcodec - set
> - */
> - XvMCSurface* p_future_surface;
> -
> - /** top/bottom field or frame
> - - application - unchanged
> - - libavcodec - set
> - */
> - unsigned int picture_structure;
> -
> - /** XVMC_SECOND_FIELD - 1st or 2nd field in the sequence
> - - application - unchanged
> - - libavcodec - set
> - */
> - unsigned int flags;
> -//}@
> -
> - /** Number of macroblock descriptions in the mv_blocks array
> - that have already been passed to the hardware.
> - - application - zeroes it on get_buffer().
> - A successful ff_draw_horiz_band() may increment it
> - with filled_mb_block_num or zero both.
> - - libavcodec - unchanged
> - */
> - int start_mv_blocks_num;
> -
> - /** Number of new macroblock descriptions in the mv_blocks array (after
> - start_mv_blocks_num) that are filled by libavcodec and have to be
> - passed to the hardware.
> - - application - zeroes it on get_buffer() or after successful
> - ff_draw_horiz_band().
> - - libavcodec - increment with one of each stored MB
> - */
> - int filled_mv_blocks_num;
> -
> - /** Number of the next free data block; one data block consists of
> - 64 short values in the data_blocks array.
> - All blocks before this one have already been claimed by placing their
> - position into the corresponding block description structure field,
> - that are part of the mv_blocks array.
> - - application - zeroes it on get_buffer().
> - A successful ff_draw_horiz_band() may zero it together
> - with start_mb_blocks_num.
> - - libavcodec - each decoded macroblock increases it by the number
> - of coded blocks it contains.
> - */
> - int next_free_data_block_num;
> -};
> -
> -/**
> - * @}
> - */
> -
> -#endif /* AVCODEC_XVMC_H */
LGTM.
_______________________________________________
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-03-07 15:21 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-07 15:04 [FFmpeg-devel] [PATCH] avcodec/xvmc: Remove header Andreas Rheinhardt
2024-03-07 15:21 ` James Almer
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