Browse Source

Don't unload plugins that use inlinedisplay

Signed-off-by: falkTX <falktx@falktx.com>
tags/v2.1-rc1
falkTX 5 years ago
parent
commit
93579acf27
Signed by: falkTX <falktx@falktx.com> GPG Key ID: CDBAA37ABC74FBA0
3 changed files with 36 additions and 9 deletions
  1. +5
    -0
      source/backend/plugin/CarlaPluginInternal.cpp
  2. +1
    -0
      source/backend/plugin/CarlaPluginInternal.hpp
  3. +30
    -9
      source/backend/plugin/CarlaPluginLV2.cpp

+ 5
- 0
source/backend/plugin/CarlaPluginInternal.cpp View File

@@ -785,6 +785,11 @@ bool CarlaPlugin::ProtectedData::libClose() noexcept
return ret; return ret;
} }


void CarlaPlugin::ProtectedData::setCanDeleteLib(const bool canDelete) noexcept
{
sLibCounter.setCanDelete(lib, canDelete);
}

bool CarlaPlugin::ProtectedData::uiLibOpen(const char* const fname, const bool canDelete) noexcept bool CarlaPlugin::ProtectedData::uiLibOpen(const char* const fname, const bool canDelete) noexcept
{ {
uiLib = sLibCounter.open(fname, canDelete); uiLib = sLibCounter.open(fname, canDelete);


+ 1
- 0
source/backend/plugin/CarlaPluginInternal.hpp View File

@@ -370,6 +370,7 @@ struct CarlaPlugin::ProtectedData {


bool libOpen(const char* const filename) noexcept; bool libOpen(const char* const filename) noexcept;
bool libClose() noexcept; bool libClose() noexcept;
void setCanDeleteLib(const bool canDelete) noexcept;


bool uiLibOpen(const char* const filename, const bool canDelete) noexcept; bool uiLibOpen(const char* const filename, const bool canDelete) noexcept;
bool uiLibClose() noexcept; bool uiLibClose() noexcept;


+ 30
- 9
source/backend/plugin/CarlaPluginLV2.cpp View File

@@ -593,6 +593,8 @@ public:
{ {
carla_debug("CarlaPluginLV2::~CarlaPluginLV2()"); carla_debug("CarlaPluginLV2::~CarlaPluginLV2()");


fInlineDisplayNeedsRedraw = false;

// close UI // close UI
if (fUI.type != UI::TYPE_NULL) if (fUI.type != UI::TYPE_NULL)
{ {
@@ -1694,16 +1696,24 @@ public:


if (fInlineDisplayNeedsRedraw) if (fInlineDisplayNeedsRedraw)
{ {
const int64_t timeNow = water::Time::currentTimeMillis();
// TESTING
CARLA_SAFE_ASSERT(pData->enabled)
CARLA_SAFE_ASSERT(!pData->engine->isAboutToClose());
CARLA_SAFE_ASSERT(pData->client->isActive());


if (timeNow - fInlineDisplayLastRedrawTime > (1000 / 30))
if (pData->enabled && !pData->engine->isAboutToClose() && pData->client->isActive())
{ {
fInlineDisplayNeedsRedraw = false;
fInlineDisplayLastRedrawTime = timeNow;
pData->engine->callback(true, true,
ENGINE_CALLBACK_INLINE_DISPLAY_REDRAW,
pData->id,
0, 0, 0, 0.0f, nullptr);
const int64_t timeNow = water::Time::currentTimeMillis();

if (timeNow - fInlineDisplayLastRedrawTime > (1000 / 30))
{
fInlineDisplayNeedsRedraw = false;
fInlineDisplayLastRedrawTime = timeNow;
pData->engine->callback(true, true,
ENGINE_CALLBACK_INLINE_DISPLAY_REDRAW,
pData->id,
0, 0, 0, 0.0f, nullptr);
}
} }
} }


@@ -4717,9 +4727,14 @@ public:
if (fExt.inlineDisplay != nullptr) if (fExt.inlineDisplay != nullptr)
{ {
if (fExt.inlineDisplay->render != nullptr) if (fExt.inlineDisplay->render != nullptr)
{
pData->hints |= PLUGIN_HAS_INLINE_DISPLAY; pData->hints |= PLUGIN_HAS_INLINE_DISPLAY;
pData->setCanDeleteLib(false);
}
else else
{
fExt.inlineDisplay = nullptr; fExt.inlineDisplay = nullptr;
}
} }
} }


@@ -5437,7 +5452,13 @@ public:
} }


// --------------------------------------------------------------- // ---------------------------------------------------------------
// get info
// set icon

if (std::strncmp(fDescriptor->URI, "http://distrho.sf.net/", 22) == 0)
pData->iconName = carla_strdup_safe("distrho");

// ---------------------------------------------------------------
// set info


if (name != nullptr && name[0] != '\0') if (name != nullptr && name[0] != '\0')
pData->name = pData->engine->getUniquePluginName(name); pData->name = pData->engine->getUniquePluginName(name);


Loading…
Cancel
Save