style/quality only

No functional changes.

- add `override' everywhere where missing
- almost pass clang's `-Wweak-vtables'
- avoid some float/double conversions
- remove unused private members
- make signedness conversions explicit
- put stuff in right namespaces to aid analysis

1 files changed, 22 insertions, 25 deletions

diff --git a/api/plugin-api.cpp b/api/plugin-api.cpp
index 404ed434..3630beae 100644
--- a/api/plugin-api.cpp
+++ b/api/plugin-api.cpp
@@ -27,35 +27,27 @@ void BaseDialog::done(int)
 
 // these exist so that vtable is emitted in a single compilation unit, not all of them.
 
+Metadata_::Metadata_() = default;
 Metadata_::~Metadata_() = default;
-IFilter::~IFilter() = default;
-IProtocol::~IProtocol() = default;
-ITracker::~ITracker() = default;
-IExtension::~IExtension() = default;
-
-void ITrackerDialog::register_tracker(ITracker*) {}
-void ITrackerDialog::unregister_tracker() {}
-
-bool ITracker::center() { return false; }
-
-module_status ITracker::status_ok()
-{
-    return module_status();
-}
-
-module_status ITracker::error(const QString& error)
-{
-    return module_status(error);
-}
+Metadata::Metadata() = default;
+Metadata::~Metadata() = default;
 
-Metadata_::Metadata_() = default;
 IFilter::IFilter() = default;
+IFilter::~IFilter() = default;
 IFilterDialog::IFilterDialog() = default;
+IFilterDialog::~IFilterDialog() = default;
 IProtocol::IProtocol() = default;
+IProtocol::~IProtocol() = default;
 IProtocolDialog::IProtocolDialog() = default;
+IProtocolDialog::~IProtocolDialog() = default;
 ITracker::ITracker() = default;
+ITracker::~ITracker() = default;
+bool ITracker::center() { return false; }
 ITrackerDialog::ITrackerDialog() = default;
-
+ITrackerDialog::~ITrackerDialog() = default;
+void ITrackerDialog::register_tracker(ITracker*) {}
+void ITrackerDialog::unregister_tracker() {}
+IExtension::~IExtension() = default;
 IExtensionDialog::~IExtensionDialog() = default;
 
 bool module_status::is_ok() const
@@ -63,8 +55,8 @@ bool module_status::is_ok() const
     return error.isNull();
 }
 
+module_status_mixin::~module_status_mixin() = default;
 module_status::module_status(QString error) : error(std::move(error)) {}
-
 module_status module_status_mixin::status_ok() { return {}; }
 
 module_status module_status_mixin::error(const QString& error)
@@ -72,7 +64,12 @@ module_status module_status_mixin::error(const QString& error)
     return module_status(error.isEmpty() ? "Unknown error" : error);
 }
 
-Metadata::Metadata() = default;
-Metadata::~Metadata() = default;
-
+module_status ITracker::status_ok()
+{
+    return module_status();
+}
 
+module_status ITracker::error(const QString& error)
+{
+    return module_status(error);
+}