pick v20 -v25

[pppanghu77]

No description provided.

[sourcery-ai]

Sorry @pppanghu77, you have reached your weekly rate limit of 500000 diff characters.

Please try again later or upgrade to continue using Sourcery

[deepin-ci-robot]

deepin pr auto review

我来对这段代码进行审查和分析：

1. text_file_saver.cpp 的修改：

• 代码安全：禁用QSaveFile可能会降低文件写入的安全性，因为QSaveFile提供了原子写入保护。建议：
  • 可以考虑添加一个配置选项让用户选择是否使用QSaveFile
  • 或者只对特定权限要求的文件禁用QSaveFile
• 代码质量：移除了过多的调试输出(qDebug)，这是好的改进
• 建议：保留文件名长度检查的逻辑，但可以简化实现

2. editwrapper.cpp 的修改：

• 代码安全：使用QPointer检查对象生命周期的改进很好，避免了潜在的悬空指针问题
• 代码质量：代码结构更清晰，错误处理更完善
• 建议：可以考虑将lambda表达式提取为单独的成员函数，提高代码可读性

3. startmanager.cpp 的修改：

• 代码质量：移除了不必要的空窗口检查，简化了逻辑
• 建议：添加的日志信息有助于调试，很好

4. window.cpp 的修改：

• 代码逻辑：
  • saveAllFiles() 改名为 saveAllFloatingFiles() 更准确地反映了函数功能
  • 改进了文件保存逻辑，特别是对草稿文件的处理
  • 打印对话框的文档名称处理更合理
• 代码安全：
  • 增加了对文件存在性的检查
  • 改进了关闭窗口时的文件保存逻辑
• 建议：
  • 可以考虑将saveAllFloatingFiles()中的保存逻辑进一步拆分成更小的函数
  • 文件路径处理可以统一使用QFileInfo

5. 整体建议：

• 错误处理：建议统一错误处理机制，比如使用统一的错误码或异常处理
• 日志：建议统一日志级别和格式
• 代码复用：一些文件操作逻辑可以提取为公共函数
• 测试：建议增加单元测试，特别是文件保存相关的功能

这些修改总体上是积极的，提高了代码的健壮性和可维护性。主要关注点是文件操作的安全性和错误处理的完整性。

[github-actions]

Note

[静态代码检查]

• 敏感词检查失败, 检测到1个文件存在敏感词

详情

{
    "src/widgets/window.cpp": [
        {
            "line": "    QString key = \"base/enable\";",
            "line_number": 389,
            "rule": "S106",
            "reason": "Var naming | 64f28539d9"
        }
    ]
}

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, pppanghu77

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

[pppanghu77]

/merge