Skip to content

sync: from linuxdeepin/dtkgui#120

Merged
18202781743 merged 1 commit intomasterfrom
sync-pr-313-nosync
Apr 29, 2025
Merged

sync: from linuxdeepin/dtkgui#120
18202781743 merged 1 commit intomasterfrom
sync-pr-313-nosync

Conversation

@deepin-ci-robot
Copy link
Copy Markdown
Contributor

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#313

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#313
@deepin-ci-robot
Copy link
Copy Markdown
Contributor Author

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

Details Needs approval from an approver in each of these files:

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

@deepin-ci-robot
Copy link
Copy Markdown
Contributor Author

deepin pr auto review

关键摘要:

  • 使用了宏定义和全局静态对象,确保这些宏定义和全局对象在多线程环境下是安全的。
  • 修改了Q_GLOBAL_STATIC的使用,增加了参数列表,这可能是为了初始化对象时提供更多的配置选项。

是否建议立即修改:

  • 否,当前的修改看起来是为了改进对象的初始化方式,没有明显的错误或问题。但是,需要确保DTK_CORE_NAMESPACE::DConfig::globalThread()DTK_CORE_NAMESPACE::DSGApplication::id()的返回值是正确的,并且这些修改不会引入新的问题。

@18202781743 18202781743 merged commit 214a63f into master Apr 29, 2025
11 of 13 checks passed
@18202781743 18202781743 deleted the sync-pr-313-nosync branch April 29, 2025 06:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants