fix coverity violations

Author: jll63

include/boost/openmethod/core.hpp

@@ -2740,6 +2740,8 @@ method<Id, ReturnType(Parameters...), Registry>::override_impl<
 #pragma GCC diagnostic ignored "-Wmaybe-uninitialized"
 #endif
 
+    // zero-initalized static variable
+    // coverity[uninit_use:FALSE]
     if (overrider_info::method) {
         BOOST_ASSERT(overrider_info::method == &fn);
         return;

include/boost/openmethod/initialize.hpp

@@ -401,7 +401,7 @@ struct registry<Policies...>::compiler : detail::generic_compiler {
     compiler(Options... opts);
 
     auto compile();
-    auto initialize();
+    void initialize();
     void install_global_tables();
 
     void augment_classes();
@@ -461,13 +461,10 @@ auto registry<Policies...>::compiler<Options...>::compile() {
 
 template<class... Policies>
 template<class... Options>
-auto registry<Policies...>::compiler<Options...>::initialize() {
+void registry<Policies...>::compiler<Options...>::initialize() {
     compile();
     install_global_tables();
-
     registry<Policies...>::initialized = true;
-
-    return *this;
 }
 
 template<class... Policies>

include/boost/openmethod/policies/default_error_handler.hpp

@@ -96,7 +96,7 @@ struct default_error_handler : error_handler {
         //! @return The previous function.
         static auto set(function_type new_handler) -> function_type {
             auto prev = handler;
-            handler = new_handler;
+            handler = std::move(new_handler);
 
             return prev;
         }

include/boost/openmethod/policies/vptr_map.hpp

@@ -98,8 +98,11 @@ class vptr_map : public vptr {
             }
 
             if constexpr (Registry::has_indirect_vptr) {
+                // check for valid iterator is done if runtime_checks is enabled
+                // coverity[deref_iterator:SUPPRESS]
                 return *iter->second;
             } else {
+                // coverity[deref_iterator:SUPPRESS]
                 return iter->second;
             }
         }