Some cleanup on Timer.

Use a different name for handler as 'updateTimer' is also used as
a method name within the same class.  Also no need to set the interval
repeatedly & no need to stop the timer when it times out.

Change-Id: I430bc26b945d179c87310eaefd9d94986192841c

chart2/source/view/charttypes/GL3DBarChart.cxx

@@ -561,7 +561,7 @@ GL3DBarChart::GL3DBarChart(
             mbAutoFly = atoi(aAutoFly);
         }
         maTimer.SetTimeout(DATA_UPDATE_TIME);
-        maTimer.SetTimeoutHdl(LINK(this, GL3DBarChart, updateTimer));
+        maTimer.SetTimeoutHdl(LINK(this, GL3DBarChart, UpdateTimerHdl));
         maTimer.Start();
         osl_getSystemTime(&maFPSRenderStartTime);
         osl_getSystemTime(&maFPSRenderEndTime);
@@ -1474,11 +1474,9 @@ void GL3DBarChart::processAutoFly(sal_uInt32 nId, sal_uInt32 nColor)
     }
 }
 
-IMPL_LINK_NOARG(GL3DBarChart, updateTimer)
+IMPL_LINK_NOARG(GL3DBarChart, UpdateTimerHdl)
 {
-    maTimer.Stop();
     updateScreenText();
-    maTimer.SetTimeout(DATA_UPDATE_TIME);
     maTimer.Start();
     return 0;
 }

chart2/source/view/inc/GL3DBarChart.hxx

@@ -101,7 +101,7 @@ private:
     void updateScreenText();
     void updateRenderFPS();
     void updateDataUpdateFPS();
-    DECL_LINK(updateTimer, void*);
+    DECL_LINK(UpdateTimerHdl, void*);
     int calcTimeInterval(TimeValue &startTime, TimeValue &endTime);
     float addScreenTextShape(OUString &nStr, glm::vec2 aLeftOrRightTop, float nTextHeight, bool bLeftTopFlag = true,
                                   const glm::vec4& rColor = glm::vec4(0.0f, 1.0f, 1.0f, 0.0f),