clang-tidy: fix override issues

This run of `clang-tidy` removes unnecessary `virtual` keywords and adds
`override` where necessary.

Applications/ParaView/ParaViewMainWindow.h

@@ -42,13 +42,13 @@ class ParaViewMainWindow : public QMainWindow
 
 public:
   ParaViewMainWindow();
-  ~ParaViewMainWindow();
+  ~ParaViewMainWindow() override;
 
 protected:
-  void dragEnterEvent(QDragEnterEvent* evt);
-  void dropEvent(QDropEvent* evt);
-  void showEvent(QShowEvent* evt);
-  void closeEvent(QCloseEvent* evt);
+  void dragEnterEvent(QDragEnterEvent* evt) override;
+  void dropEvent(QDropEvent* evt) override;
+  void showEvent(QShowEvent* evt) override;
+  void closeEvent(QCloseEvent* evt) override;
 
 protected slots:
   void showHelpForProxy(const QString& groupname, const QString& proxyname);

ParaViewCore/Animation/vtkAnimationPlayer.h

@@ -88,7 +88,7 @@ public:
 
 protected:
   vtkAnimationPlayer();
-  ~vtkAnimationPlayer();
+  ~vtkAnimationPlayer() override;
 
   friend class vtkCompositeAnimationPlayer;
 

ParaViewCore/Animation/vtkCompositeAnimationPlayer.h

@@ -73,19 +73,19 @@ public:
 
 protected:
   vtkCompositeAnimationPlayer();
-  ~vtkCompositeAnimationPlayer();
+  ~vtkCompositeAnimationPlayer() override;
 
   //@{
   /**
    * Delegated to the active animation player.
    */
-  virtual void StartLoop(double starttime, double endtime, double* playbackWindow) VTK_OVERRIDE;
-  virtual void EndLoop() VTK_OVERRIDE;
-  virtual double GetNextTime(double currentime) VTK_OVERRIDE;
+  void StartLoop(double starttime, double endtime, double* playbackWindow) VTK_OVERRIDE;
+  void EndLoop() VTK_OVERRIDE;
+  double GetNextTime(double currentime) VTK_OVERRIDE;
   //@}
 
-  virtual double GoToNext(double start, double end, double currenttime) VTK_OVERRIDE;
-  virtual double GoToPrevious(double start, double end, double currenttime) VTK_OVERRIDE;
+  double GoToNext(double start, double end, double currenttime) VTK_OVERRIDE;
+  double GoToPrevious(double start, double end, double currenttime) VTK_OVERRIDE;
 
   vtkAnimationPlayer* GetActivePlayer();
 

ParaViewCore/Animation/vtkPVAnimationCue.h

@@ -96,19 +96,19 @@ public:
   /**
    * Overridden to ignore the calls when this->Enabled == false.
    */
-  virtual void Initialize() VTK_OVERRIDE;
-  virtual void Tick(double currenttime, double deltatime, double clocktime) VTK_OVERRIDE;
-  virtual void Finalize() VTK_OVERRIDE;
+  void Initialize() VTK_OVERRIDE;
+  void Tick(double currenttime, double deltatime, double clocktime) VTK_OVERRIDE;
+  void Finalize() VTK_OVERRIDE;
   //@}
 
 protected:
   vtkPVAnimationCue();
-  ~vtkPVAnimationCue();
+  ~vtkPVAnimationCue() override;
 
   //@{
-  virtual void StartCueInternal() VTK_OVERRIDE;
-  virtual void TickInternal(double currenttime, double deltatime, double clocktime) VTK_OVERRIDE;
-  virtual void EndCueInternal() VTK_OVERRIDE;
+  void StartCueInternal() VTK_OVERRIDE;
+  void TickInternal(double currenttime, double deltatime, double clocktime) VTK_OVERRIDE;
+  void EndCueInternal() VTK_OVERRIDE;
   //@}
 
   friend class vtkSMAnimationSceneProxy;

ParaViewCore/Animation/vtkPVBooleanKeyFrame.h

@@ -35,12 +35,12 @@ public:
    * currenttime is normalized to the time range between
    * this key frame and the next key frame.
    */
-  virtual void UpdateValue(
+  void UpdateValue(
     double currenttime, vtkPVAnimationCue* cueProxy, vtkPVKeyFrame* next) VTK_OVERRIDE;
 
 protected:
   vtkPVBooleanKeyFrame();
-  ~vtkPVBooleanKeyFrame();
+  ~vtkPVBooleanKeyFrame() override;
 
 private:
   vtkPVBooleanKeyFrame(const vtkPVBooleanKeyFrame&) VTK_DELETE_FUNCTION;

ParaViewCore/Animation/vtkPVCameraAnimationCue.h

@@ -63,15 +63,15 @@ public:
   void SetTimeKeeper(vtkSMProxy*);
   //@}
 
-  virtual void BeginUpdateAnimationValues() VTK_OVERRIDE {}
-  virtual void SetAnimationValue(int, double) VTK_OVERRIDE {}
-  virtual void EndUpdateAnimationValues() VTK_OVERRIDE;
+  void BeginUpdateAnimationValues() VTK_OVERRIDE {}
+  void SetAnimationValue(int, double) VTK_OVERRIDE {}
+  void EndUpdateAnimationValues() VTK_OVERRIDE;
 
   void SetDataSourceProxy(vtkSMProxy* dataSourceProxy);
 
 protected:
   vtkPVCameraAnimationCue();
-  ~vtkPVCameraAnimationCue();
+  ~vtkPVCameraAnimationCue() override;
 
   vtkPVRenderView* View;
   vtkSMProxy* DataSourceProxy;

ParaViewCore/Animation/vtkPVCameraCueManipulator.h

@@ -73,17 +73,17 @@ public:
 
 protected:
   vtkPVCameraCueManipulator();
-  ~vtkPVCameraCueManipulator();
+  ~vtkPVCameraCueManipulator() override;
 
   int Mode;
 
-  virtual void Initialize(vtkPVAnimationCue*) VTK_OVERRIDE;
-  virtual void Finalize(vtkPVAnimationCue*) VTK_OVERRIDE;
+  void Initialize(vtkPVAnimationCue*) VTK_OVERRIDE;
+  void Finalize(vtkPVAnimationCue*) VTK_OVERRIDE;
   /**
    * This updates the values based on currenttime.
    * currenttime is normalized to the time range of the Cue.
    */
-  virtual void UpdateValue(double currenttime, vtkPVAnimationCue* cueproxy) VTK_OVERRIDE;
+  void UpdateValue(double currenttime, vtkPVAnimationCue* cueproxy) VTK_OVERRIDE;
 
   vtkCameraInterpolator* CameraInterpolator;
   vtkSMProxy* DataSourceProxy;

ParaViewCore/Animation/vtkPVCameraKeyFrame.h

@@ -43,12 +43,11 @@ public:
    * this method is called to allow the key frame to use vtkCameraInterpolator2