Frame: add isRefCounted method

Clement Champetier · Clement Champetier · commit aade71ff25e1 · 2016-10-26T12:18:23.000+02:00
Will be used when calling refFrame.
diff --git a/src/AvTranscoder/data/decoded/Frame.cpp b/src/AvTranscoder/data/decoded/Frame.cpp
@@ -68,6 +68,11 @@ void Frame::copyProperties(const Frame& otherFrame)
     av_frame_copy_props(_frame, &otherFrame.getAVFrame());
 }
 
+bool Frame::isRefCounted() const
+{
+    return _frame->buf[0];
+}
+
 void Frame::refFrame(const Frame& otherFrame)
 {
     const int ret = av_frame_ref(_frame, &otherFrame.getAVFrame());
diff --git a/src/AvTranscoder/data/decoded/Frame.hpp b/src/AvTranscoder/data/decoded/Frame.hpp
@@ -52,8 +52,10 @@ class AvExport Frame
 
     /**
      * @brief Copy the data of the given Frame.
-     * @note This function does not allocate anything: the current frame must be already initialized and
+     * This function does not allocate anything: the current frame must be already initialized and
      * allocated with the same parameters as the given frame, to be ready for memcpy instructions.
+     * @note It copies the frame data (i.e. the contents of the data / extended data arrays), not any other properties.
+     * @see copyProperties
      */
     void copyData(const Frame& frameToRef);
 
@@ -62,6 +64,11 @@ class AvExport Frame
      */
     void copyProperties(const Frame& otherFrame);
 
+    /**
+     * @brief If the data buffer of the frame refers to data allocated by an other frame.
+     */
+    bool isRefCounted() const;
+
     /**
      * @brief Copy frame properties and create a new reference to data of the given frame.
      * @warning This method allocates new data that will be freed by calling clear method or the destructor of the referenced frame.

Original file line number	Diff line number	Diff line change
`@@ -68,6 +68,11 @@ void Frame::copyProperties(const Frame& otherFrame)`
`68`	`68`	`av_frame_copy_props(_frame, &otherFrame.getAVFrame());`
`69`	`69`	`}`
`70`	`70`
	`71`	`+bool Frame::isRefCounted() const`
	`72`	`+{`
	`73`	`+ return _frame->buf[0];`
	`74`	`+}`
	`75`	`+`
`71`	`76`	`void Frame::refFrame(const Frame& otherFrame)`
`72`	`77`	`{`
`73`	`78`	`const int ret = av_frame_ref(_frame, &otherFrame.getAVFrame());`