From 4adf8a83a6860f8dd4c6bf339b894a61c3664235 Mon Sep 17 00:00:00 2001
From: Peter Barker <pbarker@barker.dropbear.id.au>
Date: Wed, 3 May 2017 07:49:30 +1000
Subject: [PATCH] AP_Notify: remove non-required initialisation of variables

These objects are dynamically allocated, and we zero the memory
as we allocate them
---
 libraries/AP_Notify/Display.cpp | 3 ---
 libraries/AP_Notify/Display.h   | 4 ++--
 2 files changed, 2 insertions(+), 5 deletions(-)

diff --git a/libraries/AP_Notify/Display.cpp b/libraries/AP_Notify/Display.cpp
index 4045f90cd1..207e990fee 100644
--- a/libraries/AP_Notify/Display.cpp
+++ b/libraries/AP_Notify/Display.cpp
@@ -324,9 +324,6 @@ bool Display::init(void)
         return true;
     }
 
-    _mstartpos = 0; // ticker shift position
-    _movedelay = 4; // ticker delay before shifting after new message displayed
-
     // initialise driver
     for(uint8_t i=0; i<8 && _driver == nullptr; i++) {
         if (! (I2C_BUS_PROBE_MASK & (1<<i))) {
diff --git a/libraries/AP_Notify/Display.h b/libraries/AP_Notify/Display.h
index 96614ee582..afa2c6a1af 100644
--- a/libraries/AP_Notify/Display.h
+++ b/libraries/AP_Notify/Display.h
@@ -34,8 +34,8 @@ private:
 
     bool _healthy;
 
-    uint8_t _mstartpos;
-    uint8_t _movedelay;
+    uint8_t _mstartpos; // ticker shift position
+    uint8_t _movedelay; // ticker delay before shifting after new message displayed
     uint8_t _screenpage;
 
     // stop showing text in display after this many millis: