Re: [Mesa-dev] [PATCH 5/5] gallium/util: don't call close() on Windows in u_tests.c

[Nicolai Hähnle]

For the series:

Reviewed-by: Nicolai Hähnle 

On 17.10.2017 23:17, Brian Paul wrote:

---
  src/gallium/auxiliary/util/u_tests.c | 2 ++
  1 file changed, 2 insertions(+)

diff --git a/src/gallium/auxiliary/util/u_tests.c 
b/src/gallium/auxiliary/util/u_tests.c
index 3cc79af..2548b46 100644
--- a/src/gallium/auxiliary/util/u_tests.c
+++ b/src/gallium/auxiliary/util/u_tests.c
@@ -567,6 +567,7 @@ test_sync_file_fences(struct pipe_context *ctx)
 pass = pass && screen->fence_finish(screen, NULL, final_fence, 0);
  
 /* Cleanup. */

+#ifndef PIPE_OS_WINDOWS
 if (buf_fd >= 0)
close(buf_fd);
 if (tex_fd >= 0)
@@ -575,6 +576,7 @@ test_sync_file_fences(struct pipe_context *ctx)
close(merged_fd);
 if (final_fd >= 0)
close(final_fd);
+#endif
  
 screen->fence_reference(screen, _fence, NULL);

 screen->fence_reference(screen, _fence, NULL);




--
Lerne, wie die Welt wirklich ist,
Aber vergiss niemals, wie sie sein sollte.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

[Mesa-dev] [PATCH 5/5] gallium/util: don't call close() on Windows in u_tests.c

[Brian Paul]

---
 src/gallium/auxiliary/util/u_tests.c | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/gallium/auxiliary/util/u_tests.c 
b/src/gallium/auxiliary/util/u_tests.c
index 3cc79af..2548b46 100644
--- a/src/gallium/auxiliary/util/u_tests.c
+++ b/src/gallium/auxiliary/util/u_tests.c
@@ -567,6 +567,7 @@ test_sync_file_fences(struct pipe_context *ctx)
pass = pass && screen->fence_finish(screen, NULL, final_fence, 0);
 
/* Cleanup. */
+#ifndef PIPE_OS_WINDOWS
if (buf_fd >= 0)
   close(buf_fd);
if (tex_fd >= 0)
@@ -575,6 +576,7 @@ test_sync_file_fences(struct pipe_context *ctx)
   close(merged_fd);
if (final_fd >= 0)
   close(final_fd);
+#endif
 
screen->fence_reference(screen, _fence, NULL);
screen->fence_reference(screen, _fence, NULL);
-- 
1.9.1

___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev