summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorPeter Powell <petpow@saberuk.com>2019-08-13 14:03:51 +0100
committerPeter Powell <petpow@saberuk.com>2019-08-13 15:02:25 +0100
commit703ba72071b970caaa7554b1e58a13def57694dc (patch)
tree6db8b7595071d6114397350ebd8611e63056779f
parentd1513c8c6064414faaa401592d463e7149a772ba (diff)
Convert LocalExtItem to a typedef and deprecate it.
-rw-r--r--include/extensible.h16
-rw-r--r--src/base.cpp11
-rw-r--r--src/modules/extra/m_geo_maxmind.cpp4
3 files changed, 8 insertions, 23 deletions
diff --git a/include/extensible.h b/include/extensible.h
index 9df6d3e18..0efb1db5f 100644
--- a/include/extensible.h
+++ b/include/extensible.h
@@ -185,21 +185,15 @@ class CoreExport ExtensionManager
ExtMap types;
};
-/** Base class for items that are NOT synchronized between servers */
-class CoreExport LocalExtItem : public ExtensionItem
-{
- public:
- LocalExtItem(const std::string& key, ExtensibleType exttype, Module* owner);
- virtual ~LocalExtItem();
- void free(Extensible* container, void* item) CXX11_OVERRIDE = 0;
-};
+/** DEPRECATED: use ExtensionItem instead. */
+typedef ExtensionItem LocalExtItem;
template <typename T, typename Del = stdalgo::defaultdeleter<T> >
-class SimpleExtItem : public LocalExtItem
+class SimpleExtItem : public ExtensionItem
{
public:
SimpleExtItem(const std::string& Key, ExtensibleType exttype, Module* parent)
- : LocalExtItem(Key, exttype, parent)
+ : ExtensionItem(Key, exttype, parent)
{
}
@@ -250,7 +244,7 @@ class CoreExport LocalStringExt : public SimpleExtItem<std::string>
void FromInternal(Extensible* container, const std::string& value) CXX11_OVERRIDE;
};
-class CoreExport LocalIntExt : public LocalExtItem
+class CoreExport LocalIntExt : public ExtensionItem
{
public:
LocalIntExt(const std::string& key, ExtensibleType exttype, Module* owner);
diff --git a/src/base.cpp b/src/base.cpp
index afd561ff2..b4ea09bc9 100644
--- a/src/base.cpp
+++ b/src/base.cpp
@@ -284,15 +284,6 @@ void ExtensionItem::unserialize(SerializeFormat format, Extensible* container, c
}
}
-LocalExtItem::LocalExtItem(const std::string& Key, ExtensibleType exttype, Module* mod)
- : ExtensionItem(Key, exttype, mod)
-{
-}
-
-LocalExtItem::~LocalExtItem()
-{
-}
-
LocalStringExt::LocalStringExt(const std::string& Key, ExtensibleType exttype, Module* Owner)
: SimpleExtItem<std::string>(Key, exttype, Owner)
{
@@ -313,7 +304,7 @@ void LocalStringExt::FromInternal(Extensible* container, const std::string& valu
}
LocalIntExt::LocalIntExt(const std::string& Key, ExtensibleType exttype, Module* mod)
- : LocalExtItem(Key, exttype, mod)
+ : ExtensionItem(Key, exttype, mod)
{
}
diff --git a/src/modules/extra/m_geo_maxmind.cpp b/src/modules/extra/m_geo_maxmind.cpp
index 767d00114..b964a393c 100644
--- a/src/modules/extra/m_geo_maxmind.cpp
+++ b/src/modules/extra/m_geo_maxmind.cpp
@@ -32,11 +32,11 @@
#include "modules/geolocation.h"
#include <maxminddb.h>
-class GeolocationExtItem : public LocalExtItem
+class GeolocationExtItem : public ExtensionItem
{
public:
GeolocationExtItem(Module* parent)
- : LocalExtItem("geolocation", ExtensionItem::EXT_USER, parent)
+ : ExtensionItem("geolocation", ExtensionItem::EXT_USER, parent)
{
}