tidy code

danielaparker · Dec 31, 2024 · c5c3da3 · c5c3da3
1 parent a8623ac
commit c5c3da3
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 2 deletions.
diff --git a/include/jsoncons_ext/jmespath/jmespath.hpp b/include/jsoncons_ext/jmespath/jmespath.hpp
@@ -809,7 +809,7 @@ namespace detail {
             bool is_right_associative_;
 
         protected:
-            ~unary_operator() = default; // virtual destructor not needed
+            virtual ~unary_operator() = default; 
         public:
             unary_operator(operator_kind oper)
                 : precedence_level_(operator_table::precedence_level(oper)), 
@@ -847,7 +847,7 @@ namespace detail {
             std::size_t precedence_level_;
             bool is_right_associative_;
         protected:
-            ~binary_operator() = default; // virtual destructor not needed
+            virtual ~binary_operator() = default; 
         public:
             binary_operator(operator_kind oper)
                 : precedence_level_(operator_table::precedence_level(oper)), 

diff --git a/include/jsoncons_ext/jsonpath/expression.hpp b/include/jsoncons_ext/jsonpath/expression.hpp
@@ -530,6 +530,8 @@ namespace detail {
               is_right_associative_(is_right_associative)
         {
         }
+
+        virtual ~binary_operator() = default;
 
         std::size_t precedence_level() const 
         {
-Original file line number
+Diff line change
@@ Expand Up / @@ -530,6 +530,8 @@ namespace detail { @@
                   is_right_associative_(is_right_associative)
             {
             }
+            virtual ~binary_operator() = default;
             std::size_t precedence_level() const
             {
@@ Expand Down @@