kwin/colormanager.cpp
Vlad Zahorodnii d70e6c2c69 Avoid destroying yet alive color devices
This is a typo that I forgot to fix, therefore not requesting code
review. After QVector::erase() has been called, we cannot use the
iterator as it will lead to undefined behavior.
2020-12-20 17:31:00 +02:00

76 lines
1.8 KiB
C++

/*
SPDX-FileCopyrightText: 2020 Vlad Zahorodnii <vlad.zahorodnii@kde.org>
SPDX-License-Identifier: GPL-2.0-or-later
*/
#include "colormanager.h"
#include "abstract_output.h"
#include "colordevice.h"
#include "main.h"
#include "platform.h"
#include "utils.h"
namespace KWin
{
KWIN_SINGLETON_FACTORY(ColorManager)
class ColorManagerPrivate
{
public:
QVector<ColorDevice *> devices;
};
ColorManager::ColorManager(QObject *parent)
: QObject(parent)
, d(new ColorManagerPrivate)
{
connect(kwinApp()->platform(), &Platform::outputEnabled, this, &ColorManager::handleOutputEnabled);
connect(kwinApp()->platform(), &Platform::outputDisabled, this, &ColorManager::handleOutputDisabled);
}
ColorManager::~ColorManager()
{
s_self = nullptr;
}
QVector<ColorDevice *> ColorManager::devices() const
{
return d->devices;
}
ColorDevice *ColorManager::findDevice(AbstractOutput *output) const
{
auto it = std::find_if(d->devices.begin(), d->devices.end(), [&output](ColorDevice *device) {
return device->output() == output;
});
if (it != d->devices.end()) {
return *it;
}
return nullptr;
}
void ColorManager::handleOutputEnabled(AbstractOutput *output)
{
ColorDevice *device = new ColorDevice(output, this);
d->devices.append(device);
emit deviceAdded(device);
}
void ColorManager::handleOutputDisabled(AbstractOutput *output)
{
auto it = std::find_if(d->devices.begin(), d->devices.end(), [&output](ColorDevice *device) {
return device->output() == output;
});
if (it == d->devices.end()) {
qCWarning(KWIN_CORE) << "Could not find any color device for output" << output;
return;
}
ColorDevice *device = *it;
d->devices.erase(it);
emit deviceRemoved(device);
delete device;
}
} // namespace KWin