From 0ca0f3e7febabba42d4494dc96bc3b5e62841dec Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Morten=20S=C3=B8rvig?= Date: Wed, 26 May 2021 12:16:56 +0200 Subject: [PATCH] =?UTF-8?q?wasm:=20don=E2=80=99t=20leak=20QWasmScreen=20me?= =?UTF-8?q?mber=20variables?= MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit m_compositor and m_eventTranslator are owned by the screen object and can be held in a unique_ptr. Pick-to: 6.2 Change-Id: I7d5e19e7ac6f762f360d6af0fc9500e6964f737e Reviewed-by: Morten Johan Sørvig --- src/plugins/platforms/wasm/qwasmscreen.cpp | 8 ++++---- src/plugins/platforms/wasm/qwasmscreen.h | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/src/plugins/platforms/wasm/qwasmscreen.cpp b/src/plugins/platforms/wasm/qwasmscreen.cpp index 7a8bb4f6867..0b11415fd3a 100644 --- a/src/plugins/platforms/wasm/qwasmscreen.cpp +++ b/src/plugins/platforms/wasm/qwasmscreen.cpp @@ -54,9 +54,9 @@ const char * QWasmScreen::m_canvasResizeObserverCallbackContextPropertyName = "d QWasmScreen::QWasmScreen(const emscripten::val &canvas) : m_canvas(canvas) + , m_compositor(new QWasmCompositor(this)) + , m_eventTranslator(new QWasmEventTranslator(this)) { - m_compositor = new QWasmCompositor(this); - m_eventTranslator = new QWasmEventTranslator(this); updateQScreenAndCanvasRenderSize(); m_canvas.call("focus"); } @@ -84,12 +84,12 @@ QWasmScreen *QWasmScreen::get(QScreen *screen) QWasmCompositor *QWasmScreen::compositor() { - return m_compositor; + return m_compositor.get(); } QWasmEventTranslator *QWasmScreen::eventTranslator() { - return m_eventTranslator; + return m_eventTranslator.get(); } emscripten::val QWasmScreen::canvas() const diff --git a/src/plugins/platforms/wasm/qwasmscreen.h b/src/plugins/platforms/wasm/qwasmscreen.h index 14d5a2f7d1f..4acf90d5150 100644 --- a/src/plugins/platforms/wasm/qwasmscreen.h +++ b/src/plugins/platforms/wasm/qwasmscreen.h @@ -86,8 +86,8 @@ public slots: private: emscripten::val m_canvas; - QWasmCompositor *m_compositor = nullptr; - QWasmEventTranslator *m_eventTranslator = nullptr; + std::unique_ptr m_compositor; + std::unique_ptr m_eventTranslator; QRect m_geometry = QRect(0, 0, 100, 100); int m_depth = 32; QImage::Format m_format = QImage::Format_RGB32;