refactor: replace #define private public hacks with template accessor…

[zccrs]

… pattern

Replace UB-prone #define private/protected public access hacks in 5 source files with a standards-compliant C++ template-based private accessor pattern, using [temp.explicit]/12 to bypass access control without undefined behavior.

Add src/util/dprivateaccessor_p.h with DtkWidgetPrivateAccessor / DtkWidgetPrivateAccessorImpl templates and supporting macros:
D_DECLARE_PRIVATE_MEMBER, D_DECLARE_PRIVATE_METHOD,
D_DECLARE_PRIVATE_CONST_METHOD, D_PRIVATE_MEMBER, D_PRIVATE_CALL

Changes per file:

• dstyleoption.cpp: access QFont::d via accessor instead of macro hack
• dapplication.cpp: remove vestigial hack (QCoreApplication::d_ptr is protected, accessible from derived class without hacks)
• dblureffectwidget.cpp: access QWidgetRepaintManager::dirtyWidgets via accessor (guarded by Qt >= 5.14)
• dlineedit.cpp: remove vestigial #define private public (no actual private member access was present)
• dtabbar.cpp: access QDragManager::m_platformDrag, QBasicDrag::cancel, QBasicDrag::m_executed_drop_action, QBasicDrag::m_eventLoop via accessors in DTabBar::stopDrag

Use ::get(tag) qualified call in macros to avoid shadowing by class-scope get() methods (e.g. DFontSizeManager::get()).

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: zccrs

The full list of commands accepted by this bot can be found here.

Details

Needs approval from an approver in each of these files:

• debian/deepin/OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[Copilot]

Pull request overview

Refactors several Qt-private access sites to remove #define private/protected public hacks by introducing a template-based private accessor helper, and updates build/CI to support the new internal header usage.

Changes:

• Added src/util/dprivateaccessor_p.h providing D_DECLARE_PRIVATE_* and D_PRIVATE_* macros for standards-based access to private/protected members.
• Updated widget sources to use the accessor pattern (or removed now-unneeded hacks) for Qt internals.
• Updated build include paths and added new CI workflows for Deepin/Arch container builds.

Reviewed changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated 6 comments.

Show a summary per file

File	Description
src/widgets/dtabbar.cpp	Replaces direct Qt-private member access in drag-stop logic with accessor macros.
src/widgets/dstyleoption.cpp	Accesses QFont::d via accessor instead of macro-based access hack.
src/widgets/dlineedit.cpp	Removes unnecessary #define private public around Qt private header include.
src/widgets/dblureffectwidget.cpp	Uses accessor to iterate QWidgetRepaintManager::dirtyWidgets (Qt ≥ 5.14).
src/widgets/dapplication.cpp	Removes Linux-only #define private public include block that’s no longer needed.
src/util/dprivateaccessor_p.h	Introduces template-based private accessor mechanism and helper macros.
src/CMakeLists.txt	Adds src/ as a private include directory to allow util/... internal includes.
.github/workflows/dtkwidget-deepin-build.yml	Adds Deepin (crimson) container CI build workflow.
.github/workflows/dtkwidget-archlinux-build.yml	Adds Arch Linux container CI build workflow.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[deepin-ci-robot]

deepin pr auto review

这份代码变更（Git Diff）主要完成了两件重要工作：

1. 新增了 CI 构建流水线：为 Arch Linux 和 Deepin 环境添加了 GitHub Actions，支持 Qt5 和 Qt6 的双版本构建。
2. 消除了 #define private public 的黑魔法：引入了基于 C++ 显式模板实例化漏洞的 DtkWidgetPrivateAccessor 机制，以合法且符合标准的方式访问 Qt 私有类成员，替代了破坏 ODR（One Definition Rule）且易导致未定义行为的宏替换。

整体来看，这是一个质量非常高的重构。但在代码规范、安全性、CI 健壮性和 C++ 模板语法细节上，仍有改进空间。以下是详细的审查意见：

---

一、 语法与逻辑

1. D_DECLARE_PRIVATE_METHOD 宏存在严重的 C++ 语法缺陷

文件: src/util/dprivateaccessor_p.h
问题: 宏定义中 RetType (ClassName::*)(__VA_ARGS__) 在传入多个参数时会导致编译错误。例如 RetType (ClassName::*)(int, float) 会被预处理器展开为两个宏参数，触发编译器报错“宏传递了过多的参数”。
改进: 使用 C++ 变参模板或借助辅助结构体将整个函数签名包裹在括号内传递。推荐使用辅助结构体，兼容性更好：

// 辅助结构体，将成员函数指针类型作为整体传递
template<typename Signature>
struct DtkMemberFuncSignature;

template<typename Ret, typename Class, typename... Args>
struct DtkMemberFuncSignature<Ret(Class::*)(Args...)> {
    using type = Ret(Class::*)(Args...);
};

template<typename Ret, typename Class, typename... Args>
struct DtkMemberFuncSignature<Ret(Class::*)(Args...) const> {
    using type = Ret(Class::*)(Args...) const;
};

// 修改后的宏
#define D_DECLARE_PRIVATE_METHOD(TagName, ClassName, MethodName, FuncSig) \
    struct TagName { \
        using MemberPtr = typename DtkMemberFuncSignature<FuncSig>::type; \
        friend MemberPtr get(TagName) noexcept; \
    }; \
    template struct DtkWidgetPrivateAccessorImpl<TagName, &ClassName::MethodName>

// 使用示例：
// D_DECLARE_PRIVATE_METHOD(QBasicDrag_cancel_tag, QBasicDrag, cancel, void (QBasicDrag::*)());

2. QBasicDrag::cancel() 方法访问方式不当

文件: src/widgets/dtabbar.cpp
问题: 原代码调用 drag->cancel(); 是直接调用类的 protected 方法。新代码改为 D_PRIVATE_CALL(*drag, QBasicDrag_cancel_tag{});，但 cancel() 在 Qt 源码中并非私有成员，而是受保护成员。对于受保护成员，派生类可以直接访问，无需动用 Private Accessor 机制，这增加了不必要的编译依赖和运行时开销。
改进: 如果 DTabBar 相关类继承自 QBasicDrag，直接调用即可；如果不继承，建议保留 Private Accessor 机制，但需确保方法确实是 private 的。如果必须访问，建议使用上面修复后的 D_DECLARE_PRIVATE_METHOD 宏。

---

二、 代码质量

1. Tag 结构体命名污染全局命名空间

文件: src/util/dprivateaccessor_p.h 及使用处
问题: 宏 D_DECLARE_PRIVATE_MEMBER 展开后的 Tag 结构体（如 QFont_d_tag, QWidgetRepaintManager_dirtyWidgets_tag）直接暴露在全局命名空间中，且命名带有随意性，容易与其他库产生符号冲突。
改进: 虽然注释中解释了为什么不能放在子命名空间（因为 friend 声明的作用域问题），但可以通过在结构体名称前加入项目专属前缀来避免冲突，例如将 QFont_d_tag 改为 Dtk_QFont_d_tag。

2. 版权年份更新为 2026 存在风险

文件: 多个 .cpp 文件
问题: SPDX 版权年份被更新为了 2026，当前尚未到达该年份，这可能是不小心的时间推算错误或自动生成工具的 Bug。
改进: 建议核实是否应为 2024，或者保留原有的截止年份加上 -2024。

---

三、 代码性能

1. Private Accessor 的运行时开销

文件: src/util/dprivateaccessor_p.h
分析: dtk_private_detail::access(Tag) 和 get(Tag) 都声明为 inline noexcept。在现代编译器（开启 -O2 或 -O3）下，这种简单的委托调用会被完全内联，最终生成的汇编代码与直接访问成员变量无异，性能损耗可忽略不计。此设计在性能上是可靠的。

---

四、 代码安全

1. CI 流水线中的供应链攻击风险

文件: .github/workflows/dtkwidget-deepin-build.yml
问题: 在 Deepin 构建步骤中，直接通过 git clone --depth=1 拉取了 dtkcommon, dtklog, dtkcore 等源码并编译安装。如果这些上游仓库被植入恶意代码，CI 环境将直接执行它们。
改进:

• 建议对这些依赖仓库的 HEAD 进行 Pin 操作（使用特定的 Commit SHA 而非默认的分支最新提交），例如：git clone --depth=1 https://github.com/linuxdeepin/dtkcommon.git <specific-commit-sha>。
• 或者使用 GitHub Actions 的可信仓库引用机制。

2. 依赖 Qt 私有 API 的兼容性风险

文件: src/widgets/dstyleoption.cpp, src/widgets/dblureffectwidget.cpp 等
问题: 虽然新方案解决了 #define private public 导致的 ODR 违反和内存布局对齐问题，但本质上仍然在访问 Qt 的私有类（如 QFontPrivate, QWidgetRepaintManager）。Qt 的小版本升级仍可能导致成员变量名变更或移除，从而引发编译失败。
改进:

• 必须在 CI 中针对 Qt5 (5.15.x) 和 Qt6 的多个小版本进行持续集成测试。
• 建议在 dprivateaccessor_p.h 的注释中强烈警告：此机制仅作为访问 Qt 私有 API 的最后手段，且需随 Qt 版本更新进行维护。

---

五、 CI/CD 流程改进

1. Arch Linux CI 中 Checkout 时机过晚

文件: .github/workflows/dtkwidget-archlinux-build.yml
问题: actions/checkout@v4 被放在了安装大量系统依赖之后。如果仓库代码本身有问题，或者依赖安装脚本需要读取仓库内的配置，会导致不必要的依赖安装时间浪费。
改进: 将 actions/checkout@v4 移至 Steps 的第一步或第二步（仅在 pacman-key --init 之后），以便尽早拉取代码。

2. Arch Linux CI 缺乏错误处理与缓存

文件: .github/workflows/dtkwidget-archlinux-build.yml
问题: pacman -Syu 和 pacman -S 在 Docker 容器中偶尔会因为网络波动或密钥环过期失败，且每次全量安装依赖耗时极长。
改进:

• 在 pacman 命令前加入重试逻辑或更新密钥环：pacman-key --populate archlinux。
• 考虑利用 GitHub Actions 的 actions/cache 对 pacman 的缓存目录 (/var/cache/pacman/pkg) 进行缓存，大幅缩短 CI 时间。

3. Deepin CI 的 apt 源配置存在潜在故障

文件: .github/workflows/dtkwidget-deepin-build.yml
问题: 强制覆盖 /etc/apt/sources.list 并使用 [trusted=yes] 跳过了 GPG 校验。且 rm -rf /etc/apt/sources.list.d 过于暴力。
改进: 建议不要删除 /etc/apt/sources.list.d，而是仅追加或覆盖主源列表；对于内部测试源，可以跳过 GPG 校验，但对于外部 mirrors.kernel.org 仍建议导入官方 GPG Key 以确保安全。

---

总结

此次重构的核心价值非常高，彻底移除了 #define private public 这个在 C++ 中臭名昭著的 Hack，提升了代码在严格编译器下的稳定性和合规性。建议优先修复 D_DECLARE_PRIVATE_METHOD 宏的变参语法缺陷，并适度调整 CI 中的步骤顺序与安全策略，即可合入主分支。

[deepin-bot]

TAG Bot

New tag: 6.7.43
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #747

[deepin-bot]

TAG Bot

New tag: 6.7.44
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #749

[18202781743]

/forcemerge

[deepin-bot]

This pr force merged! (status: blocked)