diff mbox series

[FFmpeg-devel] Remove outdated comment about AVFrame and AVOptions

Message ID 20240420133047.274226-1-ffmpeg-devel@pileofstuff.org
State New
Headers show
Series [FFmpeg-devel] Remove outdated comment about AVFrame and AVOptions | expand

Checks

Context Check Description
yinshiyou/commit_msg_loongarch64 warning The first line of the commit message must start with a context terminated by a colon and a space, for example "lavu/opt: " or "doc: ".
andriy/commit_msg_x86 warning The first line of the commit message must start with a context terminated by a colon and a space, for example "lavu/opt: " or "doc: ".
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andrew Sayers April 20, 2024, 1:29 p.m. UTC 
---

I think this is out-of-date now avcodec_get_frame_class is gone.
Happy to add docs instead if I've missed something?

 libavutil/frame.h | 3 ---
 1 file changed, 3 deletions(-)

Comments

James Almer April 20, 2024, 3:37 p.m. UTC | #1 
On 4/20/2024 10:29 AM, Andrew Sayers wrote:
> ---
> 
> I think this is out-of-date now avcodec_get_frame_class is gone.

Yes, there's no AVClass as first element in the struct.

> Happy to add docs instead if I've missed something?
> 
>   libavutil/frame.h | 3 ---
>   1 file changed, 3 deletions(-)
> 
> diff --git a/libavutil/frame.h b/libavutil/frame.h
> index 60bb966f8b..7b8d9aee08 100644
> --- a/libavutil/frame.h
> +++ b/libavutil/frame.h
> @@ -367,9 +367,6 @@ typedef struct AVRegionOfInterest {
>    *
>    * sizeof(AVFrame) is not a part of the public ABI, so new fields may be added
>    * to the end with a minor bump.
> - *
> - * Fields can be accessed through AVOptions, the name string used, matches the
> - * C structure field name for fields accessible through AVOptions.
>    */
>   typedef struct AVFrame {
>   #define AV_NUM_DATA_POINTERS 8

Will apply.
diff mbox series

Patch

diff --git a/libavutil/frame.h b/libavutil/frame.h
index 60bb966f8b..7b8d9aee08 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -367,9 +367,6 @@  typedef struct AVRegionOfInterest {
  *
  * sizeof(AVFrame) is not a part of the public ABI, so new fields may be added
  * to the end with a minor bump.
- *
- * Fields can be accessed through AVOptions, the name string used, matches the
- * C structure field name for fields accessible through AVOptions.
  */
 typedef struct AVFrame {
 #define AV_NUM_DATA_POINTERS 8