From fd1a6ff8a0e14801b1146873588f110c9db87145 Mon Sep 17 00:00:00 2001
From: Zhao Yuhang <2546789017@qq.com>
Date: 周一, 11 12月 2023 20:30:34 +0800
Subject: [PATCH] minor tweaks

---
 src/core/contexts/abstractwindowcontext.cpp |    8 ++++----
 src/core/contexts/qtwindowcontext.cpp       |    2 +-
 src/core/contexts/win32windowcontext.cpp    |    8 ++++++--
 3 files changed, 11 insertions(+), 7 deletions(-)

diff --git a/src/core/contexts/abstractwindowcontext.cpp b/src/core/contexts/abstractwindowcontext.cpp
index fa8ee74..3eb107a 100644
--- a/src/core/contexts/abstractwindowcontext.cpp
+++ b/src/core/contexts/abstractwindowcontext.cpp
@@ -172,12 +172,12 @@
         return {};
     }
 
-    static constexpr struct {
+    static constexpr const struct {
         const quint32 lilac = MAKE_RGBA_COLOR(210, 233, 189, 226);
         const quint32 pink = MAKE_RGBA_COLOR(177, 205, 190, 240);
         const quint32 tile = MAKE_RGBA_COLOR(193, 195, 211, 203);
         const quint32 azure = MAKE_RGBA_COLOR(25, 90, 190, 255);
-    } kSampleColorSet;
+    } kSampleColorSet{};
 
     void AbstractWindowContext::virtual_hook(int id, void *data) {
         switch (id) {
@@ -189,14 +189,14 @@
             case NeedsDrawBordersHook: {
                 auto &result = *reinterpret_cast<bool *>(data);
                 result = false;
-                break;
+                return;
             }
             case DrawBordersHook: {
                 auto args = reinterpret_cast<void **>(data);
                 auto &painter = *reinterpret_cast<QPainter *>(args[0]);
                 auto &rect = *reinterpret_cast<const QRect *>(args[1]);
 
-                QPen pen;
+                QPen pen{};
                 pen.setWidth(1);
 
                 // Top
diff --git a/src/core/contexts/qtwindowcontext.cpp b/src/core/contexts/qtwindowcontext.cpp
index 631c7e2..9baa24c 100644
--- a/src/core/contexts/qtwindowcontext.cpp
+++ b/src/core/contexts/qtwindowcontext.cpp
@@ -47,7 +47,7 @@
     }
 
     QString QtWindowContext::key() const {
-        return "qt";
+        return QStringLiteral("qt");
     }
 
     void QtWindowContext::virtual_hook(int id, void *data) {
diff --git a/src/core/contexts/win32windowcontext.cpp b/src/core/contexts/win32windowcontext.cpp
index efd4d59..603079c 100644
--- a/src/core/contexts/win32windowcontext.cpp
+++ b/src/core/contexts/win32windowcontext.cpp
@@ -775,10 +775,14 @@
 
     void Win32WindowContext::virtual_hook(int id, void *data) {
         switch (id) {
+            case CentralizeHook: {
+                const auto hwnd = reinterpret_cast<HWND>(m_windowHandle->winId());
+                moveToDesktopCenter(hwnd);
+                return;
+            }
             case ShowSystemMenuHook: {
                 const auto &pos = *reinterpret_cast<const QPoint *>(data);
-                auto winId = m_windowHandle->winId();
-                auto hWnd = reinterpret_cast<HWND>(winId);
+                auto hWnd = reinterpret_cast<HWND>(m_windowHandle->winId());
                 showSystemMenu2(hWnd, qpoint2point(pos), false,
                                 m_delegate->isHostSizeFixed(m_host));
                 return;

--
Gitblit v1.9.1