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

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

[Andrew Sayers]

---

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(-)

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
-- 
2.43.0

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

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

[James Almer]

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