commit f3475971c7c9f05af263674ed9b417ee33fdbfcc
author Derek Lamberti <derek.lamberti@arm.com> Mon Sep 28 16:11:50 2020 +0100
committer Matthew Bentham <matthew.bentham@arm.com> Wed Sep 30 11:12:19 2020 +0000
tree 04f25720cd1def73832ee6d7aa1ca96793fcbe2b
parent 22a4e1539aca7d39d7abc932f0a4d0b27beedf80

Quantization copy constructor

* Fix compiler implicit copy deprecation warning.
* Simplify copy operator by removing unnecessary equality comparison.
    Now just does pointer comparison instead.

Change-Id: I9ebe170c637c636919b9d8729a78449148b7f83e
Signed-off-by: Derek Lamberti <derek.lamberti@arm.com>

include/armnn/Tensor.hpp

diff --git a/include/armnn/Tensor.hpp b/include/armnn/Tensor.hpp
index 8814d89..9589874 100644
--- a/include/armnn/Tensor.hpp
+++ b/include/armnn/Tensor.hpp
@@ -229,6 +229,11 @@
             , m_Offset(EmptyOptional())
             , m_QuantizationDim(EmptyOptional()) {}
 
+        Quantization(const Quantization& other)
+            : m_Scales(other.m_Scales)
+            , m_Offset(other.m_Offset)
+            , m_QuantizationDim(other.m_QuantizationDim) {}
+
         bool operator==(const Quantization& other) const
         {
             return ((m_Scales == other.m_Scales) && (m_Offset == other.m_Offset) &&
@@ -237,7 +242,7 @@
 
         Quantization& operator=(const Quantization& other)
         {
-            if(!(*this == other))
+            if(this != &other)
             {
                 m_Scales = other.m_Scales;
                 m_Offset = other.m_Offset;