Ref T709, avoid pure virtual assert by removing the JSON ctor, which is calling a virtual function in the ctor

This commit is contained in:
Klaus Basan
2019-08-14 23:55:32 +02:00
committed by Mat Sutcliffe
parent 0b728053f0
commit 024360a5df
5 changed files with 2 additions and 17 deletions

View File

@@ -38,11 +38,6 @@ namespace BlackMisc
CXSwiftBusSettingsQtFree::CXSwiftBusSettingsQtFree()
{}
CXSwiftBusSettingsQtFree::CXSwiftBusSettingsQtFree(const std::string &json)
{
this->parseXSwiftBusString(json);
}
bool CXSwiftBusSettingsQtFree::parseXSwiftBusString(const std::string &json)
{
if (json.empty()) { return false; }

View File

@@ -28,9 +28,6 @@ namespace BlackMisc
//! Constructor.
CXSwiftBusSettingsQtFree();
//! JSON Constructor.
CXSwiftBusSettingsQtFree(const std::string &json);
//! Destructor.
~CXSwiftBusSettingsQtFree() = default;

View File

@@ -164,7 +164,8 @@ namespace XSwiftBus
void CService::setSettingsJson(const std::string &jsonString)
{
const CSettings s(jsonString);
CSettings s;
s.parseXSwiftBusString(jsonString);
this->setSettings(s);
INFO_LOG("Received settings " + s.convertToString());
}

View File

@@ -22,11 +22,6 @@ namespace XSwiftBus
// void
}
CSettings::CSettings(const std::string &json) : CXSwiftBusSettingsQtFree(json)
{
// void
}
CSettings CSettingsProvider::getSettings() const
{
std::lock_guard<std::mutex> l(m_settingsMutex);

View File

@@ -24,9 +24,6 @@ namespace XSwiftBus
//! Constructor.
CSettings();
//! JSON constructor
CSettings(const std::string &json);
protected:
//! \copydoc BlackMisc::Simulation::Settings::CXSwiftBusSettingsQtFree::objectUpdated
virtual void objectUpdated() override final {}