yukijoou/kwin
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

slide effect: remove redundant code

Browse source
This commit is contained in:
Aleix Pol 2021-05-07 01:26:33 +02:00 committed by Aleix Pol Gonzalez
parent 77091f7fcb
commit c544d4f297
2 changed files with 6 additions and 24 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

27
src/effects/slide/slide.cpp
View file

@ -32,16 +32,14 @@ SlideEffect::SlideEffect()
connect(effects, &EffectsHandler::windowDeleted,
this, &SlideEffect::windowDeleted);
connect(effects, &EffectsHandler::numberDesktopsChanged,
this, &SlideEffect::numberDesktopsChanged);
this, &SlideEffect::stop);
connect(effects, &EffectsHandler::numberScreensChanged,
this, &SlideEffect::numberScreensChanged);
this, &SlideEffect::stop);
}
SlideEffect::~SlideEffect()
{
if (m_active) {
stop();
}
stop();
}
bool SlideEffect::supported()
@ -384,6 +382,9 @@ void SlideEffect::start(int old, int current, EffectWindow *movingWindow)
void SlideEffect::stop()
{
if (!m_active) {
return;
}
const EffectWindowList windows = effects->stackingOrder();
for (EffectWindow *w : windows) {
w->setData(WindowForceBackgroundContrastRole, QVariant());
@ -435,20 +436,4 @@ void SlideEffect::windowDeleted(EffectWindow *w)
m_paintCtx.fullscreenWindows.removeAll(w);
}
void SlideEffect::numberDesktopsChanged(uint)
{
if (!m_active) {
return;
}
stop();
}
void SlideEffect::numberScreensChanged()
{
if (!m_active) {
return;
}
stop();
}
} // namespace KWin

3
src/effects/slide/slide.h
View file

@ -61,9 +61,6 @@ private Q_SLOTS:
void windowAdded(EffectWindow *w);
void windowDeleted(EffectWindow *w);
void numberDesktopsChanged(uint old);
void numberScreensChanged();
private:
QPoint desktopCoords(int id) const;
QRect desktopGeometry(int id) const;