Mesa (master): st/nine: Ignore pDirtyRegion

[GitLab Mirror]

Module: Mesa
Branch: master
Commit: dbb08255708b9005b5bb719a94ebd93194f51861
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=dbb08255708b9005b5bb719a94ebd93194f51861

Author: Axel Davy <davyaxel0 at gmail.com>
Date:   Sat May  9 13:18:42 2020 +0200

st/nine: Ignore pDirtyRegion

We supported it, but it's not much useful.
Besides it gets more complicated to handle right when
you support resizing before display.

Signed-off-by: Axel Davy <davyaxel0 at gmail.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/5015>

---

 src/gallium/frontends/nine/swapchain9.c | 6 +++++-
 1 file changed, 5 insertions(+), 1 deletion(-)

diff --git a/src/gallium/frontends/nine/swapchain9.c b/src/gallium/frontends/nine/swapchain9.c
index b9578b54600..dab6ba6aa29 100644
--- a/src/gallium/frontends/nine/swapchain9.c
+++ b/src/gallium/frontends/nine/swapchain9.c
@@ -733,6 +733,10 @@ present( struct NineSwapChain9 *This,
         This, pSourceRect, pDestRect, pDirtyRegion,
         hDestWindowOverride, (int)dwFlags, This->buffers[0]->base.resource);
 
+    /* We can choose to only update pDirtyRegion, but the backend can choose
+     * to update everything. Let's ignore */
+    (void) pDirtyRegion;
+
     if (pSourceRect) {
         DBG("pSourceRect = (%u..%u)x(%u..%u)\n",
             pSourceRect->left, pSourceRect->right,
@@ -921,7 +925,7 @@ bypass_rendering:
     if (!This->enable_threadpool) {
         This->tasks[0]=NULL;
 
-        hr = ID3DPresent_PresentBuffer(This->present, This->present_handles[0], hDestWindowOverride, pSourceRect, pDestRect ? &dest_rect : NULL, pDirtyRegion, dwFlags);
+        hr = ID3DPresent_PresentBuffer(This->present, This->present_handles[0], hDestWindowOverride, pSourceRect, pDestRect ? &dest_rect : NULL, NULL, dwFlags);
 
         if (FAILED(hr)) { UNTESTED(3);return hr; }
     }