commit c1294faa1d60274aa14567338fc2a449c70c1801
author Sanghoon Lee <sanghoon.lee@arm.com> Fri Nov 17 11:47:41 2017 +0000
committer Anthony Barbier <anthony.barbier@arm.com> Fri Nov 02 16:35:24 2018 +0000
tree 684bbe751ab80a3fea065a44d3bb87804fe1c9b1
parent 41caa625231c24533a514606bbf2683f7d4964ad

COMPMID-494: Fixing values to const

Change-Id: I21a4191b032df17611e01d20d89cc21e8eaf68fc
Reviewed-on: https://eu-gerrit-1.euhpc.arm.com/109605
Tested-by: BSG Visual Compute Jenkins server to access repositories on http://mpd-gerrit.cambridge.arm.com <bsgcomp@arm.com>
Reviewed-by: Anthony Barbier <anthony.barbier@arm.com>

tests/validation/CL/BatchNormalizationLayer.cpp

diff --git a/tests/validation/CL/BatchNormalizationLayer.cpp b/tests/validation/CL/BatchNormalizationLayer.cpp
index c29a400..6884131 100644
--- a/tests/validation/CL/BatchNormalizationLayer.cpp
+++ b/tests/validation/CL/BatchNormalizationLayer.cpp
@@ -59,7 +59,7 @@
                shape0, shape1, epsilon, dt)
 {
     // Set fixed point position data type allowed
-    int fixed_point_position = (arm_compute::is_data_type_fixed_point(dt)) ? 3 : 0;
+    const int fixed_point_position = (arm_compute::is_data_type_fixed_point(dt)) ? 3 : 0;
 
     // Create tensors
     CLTensor src   = create_tensor<CLTensor>(shape0, dt, 1, fixed_point_position);

tests/validation/CPP/BatchNormalizationLayer.cpp

diff --git a/tests/validation/CPP/BatchNormalizationLayer.cpp b/tests/validation/CPP/BatchNormalizationLayer.cpp
index 37e2d55..e4446d1 100644
--- a/tests/validation/CPP/BatchNormalizationLayer.cpp
+++ b/tests/validation/CPP/BatchNormalizationLayer.cpp
@@ -44,7 +44,7 @@
     const auto cols       = static_cast<int>(src.shape()[0]);
     const auto rows       = static_cast<int>(src.shape()[1]);
     const auto depth      = static_cast<int>(src.shape()[2]);
-    int        upper_dims = src.shape().total_size() / (cols * rows * depth);
+    const int  upper_dims = src.shape().total_size() / (cols * rows * depth);
 
     for(int r = 0; r < upper_dims; ++r)
     {
@@ -88,7 +88,7 @@
     const auto cols       = static_cast<int>(src.shape()[0]);
     const auto rows       = static_cast<int>(src.shape()[1]);
     const auto depth      = static_cast<int>(src.shape()[2]);
-    int        upper_dims = src.shape().total_size() / (cols * rows * depth);
+    const int  upper_dims = src.shape().total_size() / (cols * rows * depth);
 
     for(int r = 0; r < upper_dims; ++r)
     {

tests/validation/Helpers.h

diff --git a/tests/validation/Helpers.h b/tests/validation/Helpers.h
index 6b1c4b9..b5e156c 100644
--- a/tests/validation/Helpers.h
+++ b/tests/validation/Helpers.h
@@ -186,7 +186,7 @@
 template <typename T>
 std::pair<T, T> get_batchnormalization_layer_test_bounds(int fixed_point_position = 1)
 {
-    bool is_float = std::is_floating_point<T>::value;
+    const bool is_float = std::is_floating_point<T>::value;
     std::pair<T, T> bounds;
 
     // Set initial values

tests/validation/NEON/BatchNormalizationLayer.cpp

diff --git a/tests/validation/NEON/BatchNormalizationLayer.cpp b/tests/validation/NEON/BatchNormalizationLayer.cpp
index b401f55..a1421d0 100644
--- a/tests/validation/NEON/BatchNormalizationLayer.cpp
+++ b/tests/validation/NEON/BatchNormalizationLayer.cpp
@@ -61,7 +61,7 @@
                shape0, shape1, epsilon, dt)
 {
     // Set fixed point position data type allowed
-    int fixed_point_position = (arm_compute::is_data_type_fixed_point(dt)) ? 3 : 0;
+    const int fixed_point_position = (arm_compute::is_data_type_fixed_point(dt)) ? 3 : 0;
 
     // Create tensors
     Tensor src   = create_tensor<Tensor>(shape0, dt, 1, fixed_point_position);