sync: from linuxdeepin/dtkgui

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#309

[deepin-ci-robot]

[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:

• 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

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 在dguiapplicationhelper.cpp文件中，Q_GLOBAL_STATIC宏的使用方式有误。Q_GLOBAL_STATIC宏需要两个参数：第一个是静态变量的类型，第二个是静态变量的名称。在代码中，_d_dconfig被定义为Q_GLOBAL_STATIC_WITH_ARGS，这可能是为了传递额外的参数给静态变量的构造函数。但是，Q_GLOBAL_STATIC_WITH_ARGS宏的使用方式不正确，应该使用Q_GLOBAL_STATIC宏，并在静态变量的定义中直接传递参数。

2. 在dguiapplicationhelper.cpp文件中，_d_dconfig的连接信号槽的方式有误。在连接信号槽时，应该使用QObject::connect函数，而不是operator()函数。正确的连接方式应该是QObject::connect(_d_dconfig, &OrgDeepinDTKPreference::themeTypeChanged, _d_dconfig, [applyThemeType] { ... });。

3. 在dplatformtheme.cpp文件中，dtkPreferenceConfig的定义方式有误。应该使用OrgDeepinDTKPreference::createGeneric函数来创建dtkPreferenceConfig，而不是直接使用DConfig::createGeneric函数。

4. 在dplatformtheme.cpp文件中，onDtkPreferenceDConfigChanged函数被注释掉了，但是这个函数在dplatformtheme_p.h文件中被声明为public slots。如果这个函数不再需要，应该将其从dplatformtheme_p.h文件中删除。

5. 在orgdeepindtkpreference.hpp文件中，OrgDeepinDTKPreference类的构造函数中使用了QMetaObject::invokeMethod函数来在另一个线程中创建DConfig实例。这可能会导致线程安全问题，应该使用QThread::create函数来创建一个新的线程，并在新线程中执行DConfig的创建。

6. 在orgdeepindtkpreference.hpp文件中，OrgDeepinDTKPreference类的initializeInConfigThread函数中使用了QMetaObject::invokeMethod函数来在另一个线程中更新OrgDeepinDTKPreference的属性。这可能会导致线程安全问题，应该使用QThread::create函数来创建一个新的线程，并在新线程中执行属性的更新。

7. 在orgdeepindtkpreference.hpp文件中，OrgDeepinDTKPreference类的updateValue函数中使用了QMetaObject::invokeMethod函数来在另一个线程中更新OrgDeepinDTKPreference的属性。这可能会导致线程安全问题，应该使用QThread::create函数来创建一个新的线程，并在新线程中执行属性的更新。

8. 在xdgiconproxyengine.cpp文件中，XdgIconProxyEngine::pixmapByEntry函数中使用了entry->pixmap函数来获取图标。但是，这个函数在QIconLoaderEngineEntry类中并没有定义。应该使用entry->icon函数来获取图标。

以上是代码审查的意见，希望能够对您有所帮助。

[deepin-bot]

TAG Bot

New tag: 6.0.33
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #117

[deepin-bot]

TAG Bot

New tag: 6.0.34
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #119