[PATCH v3 03/10] winegstreamer: Introduce a new sample_needs_copy_buffer helper.

[Rémi Bernon]

From: Rémi Bernon

---
 dlls/winegstreamer/wg_transform.c | 21 ++++++++++++++-------
 1 file changed, 14 insertions(+), 7 deletions(-)

diff --git a/dlls/winegstreamer/wg_transform.c b/dlls/winegstreamer/wg_transform.c
index 51706a6e6a6..4152856ec5a 100644
--- a/dlls/winegstreamer/wg_transform.c
+++ b/dlls/winegstreamer/wg_transform.c
@@ -755,13 +755,10 @@ static void sample_set_flags_from_buffer(struct wg_sample *sample, GstBuffer *bu
         sample->flags |= WG_SAMPLE_FLAG_DISCONTINUITY;
 }

-static NTSTATUS read_transform_output_data(GstBuffer *buffer, GstCaps *caps, gsize plane_align,
-        struct wg_sample *sample)
+static bool sample_needs_copy_buffer(struct wg_sample *sample, GstBuffer *buffer, gsize *total_size)
 {
-    gsize total_size;
-    bool needs_copy;
-    NTSTATUS status;
     GstMapInfo info;
+    bool needs_copy;

if (!gst_buffer_map(buffer, &info, GST_MAP_READ))
     {
@@ -770,10 +767,20 @@ static NTSTATUS read_transform_output_data(GstBuffer *buffer, GstCaps *caps, gsi
         return STATUS_UNSUCCESSFUL;
     }
     needs_copy = info.data != wg_sample_data(sample);
-    total_size = sample->size = info.size;
+    *total_size = sample->size = info.size;
     gst_buffer_unmap(buffer, &info);

-    if (!needs_copy)
+    return needs_copy;
+}
+
+static NTSTATUS read_transform_output_data(GstBuffer *buffer, GstCaps *caps, gsize plane_align,
+        struct wg_sample *sample)
+{
+    gsize total_size;
+    NTSTATUS status;
+    bool needs_copy;
+
+    if (!(needs_copy = sample_needs_copy_buffer(sample, buffer, &total_size)))
         status = STATUS_SUCCESS;
     else if (stream_type_from_caps(caps) == GST_STREAM_TYPE_VIDEO)
         status = copy_video_buffer(buffer, caps, plane_align, sample, &total_size);