sync: from linuxdeepin/dtkgui

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#347

[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

这段代码是一个CMake构建脚本，主要处理DTK GUI库的依赖关系。我对这段代码进行了审查，以下是改进意见：

1. 语法逻辑

• 代码逻辑基本正确，但嵌套的if语句结构可以更清晰
• if()语句应该总是使用括号来包含条件，即使条件为空
• message(WARNING ...)的格式可以更规范，添加适当的空格

2. 代码质量

• 变量命名规范，使用了大写和下划线的组合，符合CMake的命名惯例
• 注释清晰，解释了代码的目的
• 代码缩进一致，可读性好

3. 代码性能

• 此段代码是构建配置，不影响运行时性能，无需优化

4. 代码安全

• 变量设置使用了set()命令，是安全的
• 使用了REQUIRED参数确保依赖存在，提高了构建的可靠性

改进建议

# Only use libxdg under Qt5
if(NOT DTK_DISABLE_LIBXDG)
    if(${QT_VERSION_MAJOR} STREQUAL "5")
        find_package(Qt5XdgIconLoader REQUIRED)
        if(NOT Qt5XdgIconLoader_FOUND)
            message(WARNING "XdgIconLoader not found, disabling LIBXDG")
            set(DTK_DISABLE_LIBXDG ON)
        endif()
    else()
        set(DTK_DISABLE_LIBXDG ON)
    endif()
endif()

具体改进点：

1. 调整了代码缩进，使嵌套结构更清晰
2. 规范了警告消息的格式，添加了适当的空格
3. 移除了不必要的空行，使代码更紧凑
4. 保持了原有的逻辑不变，但提高了可读性
5. 确保了所有if()语句都有对应的endif()

这些改进不会改变代码的功能，但会使代码更易于维护和理解。