sync: from linuxdeepin/dtkgui

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#255

[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-bot]

TAG Bot

New tag: 6.0.21
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #62

[deepin-bot]

TAG Bot

New tag: 6.0.22
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #76

[deepin-bot]

TAG Bot

New tag: 6.0.23
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #79

[deepin-bot]

TAG Bot

New tag: 6.0.24
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #92

[deepin-ci-robot]

deepin pr auto review

代码审查意见：

1. 代码重复：

   • 在DBuiltinIconEngine和DDciIconEngine类中，availableSizes函数的实现非常相似。建议将公共逻辑提取到一个单独的函数中，以减少代码重复。

2. 默认参数：

   • 在availableSizes函数中，mode和state参数有默认值。虽然这在某些情况下可能是有用的，但最好在文档中明确说明这些参数的默认值，以便其他开发者理解。

3. 性能优化：

   • 在DBuiltinIconEngine和DDciIconEngine的availableSizes函数中，使用了reserve方法来预分配QList的大小。这是一个好的做法，因为它可以减少在列表增长时需要重新分配内存的次数。

4. 错误处理：

   • 在availableSizes函数中，没有对m_info.entries或m_dciIcon.availableSizes返回的值进行错误检查。如果这些值可能为空或无效，应该添加相应的错误处理逻辑。

5. 代码注释：

   • 在availableSizes函数中，应该添加注释来解释函数的目的、参数和返回值。这有助于其他开发者理解代码的意图。

6. 版本控制：

   • 在DBuiltinIconEngine和DDciIconEngine类中，availableSizes函数的实现使用了#if QT_VERSION >= QT_VERSION_CHECK(6, 0, 0)宏来区分不同版本的Qt。这是一个好的做法，但应该确保在文档中说明这些宏的使用目的和影响。

7. 命名一致性：

   • 在DBuiltinIconEngine和DDciIconEngine类中，availableSizes函数的命名是一致的，这很好。但是，如果这些函数在其他地方也有类似的实现，建议使用相同的命名约定，以提高代码的可读性和一致性。

综上所述，代码总体上质量较高，但可以通过上述建议进行改进，以提高代码的可维护性和可读性。