diff mbox series

[FFmpeg-devel,1/2] lavu/frame: drop mentions of non-refcounted frames

Message ID 20211125153315.26506-1-anton@khirnov.net
State Accepted
Commit 3b8efec3c58ab1e30d72427d3025a0641b442f71
Headers show
Series [FFmpeg-devel,1/2] lavu/frame: drop mentions of non-refcounted frames | expand

Checks

Context Check Description
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished
andriy/make_ppc success Make finished
andriy/make_fate_ppc success Make fate finished

Commit Message

Anton Khirnov Nov. 25, 2021, 3:33 p.m. UTC 
All frames we deal with should always be refcounted now.
---
 libavutil/frame.h | 11 +++++++----
 1 file changed, 7 insertions(+), 4 deletions(-)

Comments

Anton Khirnov Dec. 4, 2021, 1:36 p.m. UTC | #1 
pushed
diff mbox series

Patch

diff --git a/libavutil/frame.h b/libavutil/frame.h
index 753234792e..f7be2340fd 100644
--- a/libavutil/frame.h
+++ b/libavutil/frame.h
@@ -314,6 +314,9 @@  typedef struct AVFrame {
      * This might be different from the first allocated byte. For video,
      * it could even point to the end of the image data.
      *
+     * All pointers in data and extended_data must point into one of the
+     * AVBufferRef in buf or extended_buf.
+     *
      * Some decoders access areas outside 0,0 - width,height, please
      * see avcodec_align_dimensions2(). Some filters and swscale can read
      * up to 16 bytes beyond the planes, if these filters are to be used,
@@ -482,10 +485,10 @@  typedef struct AVFrame {
     uint64_t channel_layout;
 
     /**
-     * AVBuffer references backing the data for this frame. If all elements of
-     * this array are NULL, then this frame is not reference counted. This array
-     * must be filled contiguously -- if buf[i] is non-NULL then buf[j] must
-     * also be non-NULL for all j < i.
+     * AVBuffer references backing the data for this frame. All the pointers in
+     * data and extended_data must point inside one of the buffers in buf or
+     * extended_buf. This array must be filled contiguously -- if buf[i] is
+     * non-NULL then buf[j] must also be non-NULL for all j < i.
      *
      * There may be at most one AVBuffer per data plane, so for video this array
      * always contains all the references. For planar audio with more than