From 45d5328457c4c304388928a00d9b45ca563cf4a0 Mon Sep 17 00:00:00 2001
From: David Wisth <davidwisth@gmail.com>
Date: Wed, 8 Dec 2021 21:12:46 +0000
Subject: [PATCH] remove "virtual" keyword for functions with "override"

---
 gtsam_unstable/slam/PoseToPointFactor.h | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/gtsam_unstable/slam/PoseToPointFactor.h b/gtsam_unstable/slam/PoseToPointFactor.h
index be73b9312..cab48e506 100644
--- a/gtsam_unstable/slam/PoseToPointFactor.h
+++ b/gtsam_unstable/slam/PoseToPointFactor.h
@@ -45,9 +45,8 @@ class PoseToPointFactor : public NoiseModelFactor2<POSE, POINT> {
   /** implement functions needed for Testable */
 
   /** print */
-  virtual void print(
-      const std::string& s,
-      const KeyFormatter& keyFormatter = DefaultKeyFormatter) const override {
+  void print(const std::string& s, const KeyFormatter& keyFormatter =
+                                       DefaultKeyFormatter) const override {
     std::cout << s << "PoseToPointFactor(" << keyFormatter(this->key1()) << ","
               << keyFormatter(this->key2()) << ")\n"
               << "  measured: " << measured_.transpose() << std::endl;
@@ -55,8 +54,8 @@ class PoseToPointFactor : public NoiseModelFactor2<POSE, POINT> {
   }
 
   /** equals */
-  virtual bool equals(const NonlinearFactor& expected,
-                      double tol = 1e-9) const override {
+  bool equals(const NonlinearFactor& expected,
+              double tol = 1e-9) const override {
     const This* e = dynamic_cast<const This*>(&expected);
     return e != nullptr && Base::equals(*e, tol) &&
            traits<POINT>::Equals(this->measured_, e->measured_, tol);