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

x11: Pass the output name to X11Output constructor

Browse source 
The output name is static, i.e. it cannot change, so there's no point
for having a setter.
This commit is contained in:
Vlad Zahorodnii 2021-08-26 09:34:39 +03:00
parent a8810e0c33
commit bd6d04b417
3 changed files with 4 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

8
src/plugins/platforms/x11/standalone/x11_output.cpp
View file

@ -12,8 +12,9 @@
namespace KWin
{
X11Output::X11Output(QObject *parent)
X11Output::X11Output(const QString &name, QObject *parent)
: AbstractOutput(parent)
, m_name(name)
{
}
@ -22,11 +23,6 @@ QString X11Output::name() const
return m_name;
}
void X11Output::setName(QString set)
{
m_name = set;
}
QRect X11Output::geometry() const
{
if (m_geometry.isValid()) {

4
src/plugins/platforms/x11/standalone/x11_output.h
View file

@ -28,11 +28,9 @@ class KWIN_EXPORT X11Output : public AbstractOutput
Q_OBJECT
public:
explicit X11Output(QObject *parent = nullptr);
~X11Output() override = default;
explicit X11Output(const QString &name, QObject *parent = nullptr);
QString name() const override;
void setName(QString set);
QRect geometry() const override;
void setGeometry(QRect set);

3
src/plugins/platforms/x11/standalone/x11_platform.cpp
View file

@ -520,8 +520,7 @@ void X11StandalonePlatform::doUpdateOutputs()
changed.append(output);
removed.removeOne(output);
} else {
output = new X11Output();
output->setName(outputInfo.name());
output = new X11Output(outputInfo.name());
added.append(output);
}