From 01a4fabf9ee2b8eb2a7304cc2c83ac8e9f807d28 Mon Sep 17 00:00:00 2001
From: Andrew Tridgell <tridge@samba.org>
Date: Sun, 21 Apr 2013 22:23:46 +1000
Subject: [PATCH] AP_Param: make things a bit more efficient

---
 libraries/AP_Param/AP_Param.h | 16 ++--------------
 1 file changed, 2 insertions(+), 14 deletions(-)

diff --git a/libraries/AP_Param/AP_Param.h b/libraries/AP_Param/AP_Param.h
index 12f586efc2..d54d21112c 100644
--- a/libraries/AP_Param/AP_Param.h
+++ b/libraries/AP_Param/AP_Param.h
@@ -438,7 +438,7 @@ public:
 
     /// Value getter
     ///
-    T        get(void) const {
+    const T &get(void) const {
         return _value;
     }
 
@@ -459,16 +459,10 @@ public:
     ///
     /// This allows the class to be used in many situations where the value would be legal.
     ///
-    operator T &() {
+    operator const T &() const {
         return _value;
     }
 
-    /// Copy assignment from self does nothing.
-    ///
-    AP_ParamV<T,PT>& operator        =(AP_ParamV<T,PT>& v) {
-        return v;
-    }
-
     /// Copy assignment from T is equivalent to ::set.
     ///
     AP_ParamV<T,PT>& operator        =(T v) {
@@ -531,12 +525,6 @@ public:
         }
     }
 
-    /// Copy assignment from self does nothing.
-    ///
-    AP_ParamA<T,N,PT>& operator= (AP_ParamA<T,N,PT>& v) {
-        return v;
-    }
-
 protected:
     T _value[N];
 };