Mesa (master): anv/cmd_buffer: Simplify range member assignment

Nanley Chery nchery at kemper.freedesktop.org
Fri Jul 15 17:38:09 UTC 2016


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

Author: Nanley Chery <nanley.g.chery at intel.com>
Date:   Tue Jul 12 08:10:18 2016 -0700

anv/cmd_buffer: Simplify range member assignment

A ternary is clearer because the range member is assigned one of two values
dependant on one condition.

Signed-off-by: Nanley Chery <nanley.g.chery at intel.com>
Reviewed-by: Jason Ekstrand <jason at jlekstrand.net>

---

 src/intel/vulkan/anv_cmd_buffer.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

diff --git a/src/intel/vulkan/anv_cmd_buffer.c b/src/intel/vulkan/anv_cmd_buffer.c
index 9ceb194..6256df8 100644
--- a/src/intel/vulkan/anv_cmd_buffer.c
+++ b/src/intel/vulkan/anv_cmd_buffer.c
@@ -646,11 +646,9 @@ void anv_CmdBindDescriptorSets(
 
                unsigned array_size = set_layout->binding[b].array_size;
                for (unsigned j = 0; j < array_size; j++) {
-                  uint32_t range = 0;
-                  if (desc->buffer_view)
-                     range = desc->buffer_view->range;
                   push->dynamic[d].offset = *(offsets++);
-                  push->dynamic[d].range = range;
+                  push->dynamic[d].range = (desc->buffer_view) ?
+                                            desc->buffer_view->range : 0;
                   desc++;
                   d++;
                }




More information about the mesa-commit mailing list