sync: from linuxdeepin/dtkwidget

[deepin-ci-robot]

Synchronize source files from linuxdeepin/dtkwidget.

Source-pull-request: linuxdeepin/dtkwidget#693

[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

我来对这个 diff 变更进行代码审查：

1. 语法逻辑：

• 代码语法正确，在 QEvent::FontChange 事件中调用 updateSize() 是合理的。
• 保持了原有的代码结构，在 else-if 分支中处理字体变化事件。

2. 代码质量：

• 建议添加注释说明为什么在字体变化时需要更新大小。例如：

    } else if (event->type() == QEvent::FontChange) {
        // 字体变化可能导致对话框内容大小改变，需要重新计算尺寸
        d->updateSize();
    }

3. 代码性能：

• updateSize() 可能涉及布局重计算，在字体变化时调用是必要的。
• 没有明显的性能问题，因为字体变化是一个相对低频的事件。

4. 代码安全：

• 代码安全，没有使用不安全的操作。
• d指针的使用是安全的，符合Qt的设计模式。

改进建议：

1. 添加注释说明处理逻辑
2. 可以考虑将 updateSize() 的调用封装在一个单独的函数中，提高代码复用性。例如：

void DDialog::handleSizeUpdateEvent()
{
    d->updateSize();
}

void DDialog::changeEvent(QEvent *event)
{
    if (event->type() == QEvent::PaletteChange) {
        // ...
        handleSizeUpdateEvent();
    } else if (event->type() == QEvent::FontChange) {
        // 字体变化可能导致对话框内容大小改变
        handleSizeUpdateEvent();
    }
    return DAbstractDialog::changeEvent(event);
}

总的来说，这个变更是一个合理的改进，但建议添加适当的注释来提高代码的可维护性。