diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2019-05-23 16:41:14 +0200 |
---|---|---|
committer | Michael Weghorn <m.weghorn@posteo.de> | 2019-05-24 03:07:15 +0200 |
commit | 6c817f00dbcdf4e38bf22741111fb49fb6815671 (patch) | |
tree | b8e960d33b966f5983d2aa236f2f8dd0fc0791d6 /vcl/qt5 | |
parent | Resolves: tdf#125128 set Tables as "above" by default (diff) | |
download | core-6c817f00dbcdf4e38bf22741111fb49fb6815671.tar.gz core-6c817f00dbcdf4e38bf22741111fb49fb6815671.zip |
qt5: Fix build
The utility constructor using 'StockImage::Yes' was only introduced
in master commit 0f104bf33530467380044b9eb4cd3f8bd9c283f1 and is
therefore not available in libreoffice-6-2 branch.
Build was therefore broken by commit
f10c1c648e698b48dfaa679a4909b3d9de6ed868 ("tdf#123549 Qt5 implement
Qt5Menu::ShowCloseButton"), which first went unnoticed since
qt5/kde5 wasn't enabled for CI builds on branch libreoffice-6-2.
(This will be changed by
https://gerrit.libreoffice.org/#/c/72741/ .)
Also, add a missing include that lead to kde5-enabled CI builds
failing [1], which I couldn't reproduce locally
("shell/source/backends/kde5be/kde5backend.cxx:190:5: error:
‘unique_ptr’ is not a member of ‘std’").
And fix more issues showing up in [2] and follow-up builds
of the change that enables kde5 on the 6.2 branch [3].
[1] https://ci.libreoffice.org/job/gerrit_62/1535/
[2] https://ci.libreoffice.org/job/gerrit_62/1540/
[3] https://gerrit.libreoffice.org/#/c/72741/
Change-Id: I6a39a99114d15808b790242c96d0204916a0cc40
Reviewed-on: https://gerrit.libreoffice.org/72779
Reviewed-by: Michael Stahl <Michael.Stahl@cib.de>
Tested-by: Jenkins
Diffstat (limited to 'vcl/qt5')
-rw-r--r-- | vcl/qt5/Qt5Frame.cxx | 2 | ||||
-rw-r--r-- | vcl/qt5/Qt5Instance.cxx | 3 | ||||
-rw-r--r-- | vcl/qt5/Qt5Menu.cxx | 4 |
3 files changed, 5 insertions, 4 deletions
diff --git a/vcl/qt5/Qt5Frame.cxx b/vcl/qt5/Qt5Frame.cxx index 09c9afbdac29..9f8be0ad27c8 100644 --- a/vcl/qt5/Qt5Frame.cxx +++ b/vcl/qt5/Qt5Frame.cxx @@ -575,7 +575,7 @@ void Qt5Frame::SetWindowState(const SalFrameState* pState) { if (pState->mnState & WindowStateState::Maximized) SetWindowStateImpl(Qt::WindowMaximized); - else if ((pState->mnState & WindowStateState::Minimized)) + else if (pState->mnState & WindowStateState::Minimized) SetWindowStateImpl(Qt::WindowMinimized); else SetWindowStateImpl(Qt::WindowNoState); diff --git a/vcl/qt5/Qt5Instance.cxx b/vcl/qt5/Qt5Instance.cxx index c383f8600108..5ed124d07440 100644 --- a/vcl/qt5/Qt5Instance.cxx +++ b/vcl/qt5/Qt5Instance.cxx @@ -50,6 +50,7 @@ #include <headless/svpbmp.hxx> +#include <o3tl/make_unique.hxx> #include <mutex> #include <condition_variable> @@ -201,7 +202,7 @@ void Qt5Instance::ImplRunInMain() } Qt5Instance::Qt5Instance(bool bUseCairo) - : SalGenericInstance(std::make_unique<Qt5YieldMutex>()) + : SalGenericInstance(o3tl::make_unique<Qt5YieldMutex>()) , m_postUserEventId(-1) , m_bUseCairo(bUseCairo) { diff --git a/vcl/qt5/Qt5Menu.cxx b/vcl/qt5/Qt5Menu.cxx index 5c413cfd7000..47613988beac 100644 --- a/vcl/qt5/Qt5Menu.cxx +++ b/vcl/qt5/Qt5Menu.cxx @@ -617,8 +617,8 @@ void Qt5Menu::ShowCloseButton(bool bShow) if (QIcon::hasThemeIcon("window-close-symbolic")) aIcon = QIcon::fromTheme("window-close-symbolic"); else - aIcon = QIcon( - QPixmap::fromImage((toQImage(Image(StockImage::Yes, SV_RESID_BITMAP_CLOSEDOC))))); + aIcon = QIcon(QPixmap::fromImage(( + toQImage(Image(OUString("private:graphicrepository/" SV_RESID_BITMAP_CLOSEDOC)))))); pButton = new QPushButton(mpQMenuBar); pButton->setIcon(aIcon); pButton->setFlat(true); |