From 1294c280586de4b8e9568cdccaffff17f5159289 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Sebastian=20Dr=C3=B6ge?= <sebastian@centricular.com>
Date: Tue, 16 Jan 2018 18:55:25 +0200
Subject: [PATCH] Add VideoFrameRef::copy_to_ref() and ::copy_plane_to_ref()

These take a VideoFrameRef instead of a VideoFrame as destination. Next
time we break API, the existing ::copy() and ::copy_plane() functions
should be modified.
---
 gstreamer-video/src/video_frame.rs | 28 ++++++++++++++++++++++++++++
 1 file changed, 28 insertions(+)

diff --git a/gstreamer-video/src/video_frame.rs b/gstreamer-video/src/video_frame.rs
index 14f0345c2..55ceba3e8 100644
--- a/gstreamer-video/src/video_frame.rs
+++ b/gstreamer-video/src/video_frame.rs
@@ -423,6 +423,34 @@ impl<'a> VideoFrameRef<&'a gst::BufferRef> {
         }
     }
 
+    pub fn copy_to_ref(&self, dest: &mut VideoFrameRef<&mut gst::BufferRef>) -> Result<(), glib::BoolError> {
+        unsafe {
+            let res: bool = from_glib(ffi::gst_video_frame_copy(&mut dest.0, &self.0));
+            if res {
+                Ok(())
+            } else {
+                Err(glib::BoolError("Failed to copy video frame"))
+            }
+        }
+    }
+
+    pub fn copy_plane_to_ref(
+        &self,
+        dest: &mut VideoFrameRef<&mut gst::BufferRef>,
+        plane: u32,
+    ) -> Result<(), glib::BoolError> {
+        skip_assert_initialized!();
+
+        unsafe {
+            let res: bool = from_glib(ffi::gst_video_frame_copy_plane(&mut dest.0, &self.0, plane));
+            if res {
+                Ok(())
+            } else {
+                Err(glib::BoolError("Failed to copy video frame plane"))
+            }
+        }
+    }
+
     pub fn format(&self) -> ::VideoFormat {
         self.info().format()
     }