fix(ui): scale large images and preserve original dimensions

[add-uos]

fix(ui): scale large images and preserve original dimensions

build(cmake): improve Qt version detection logic

[deepin-ci-robot]

deepin pr auto review

我来分析一下这个diff的代码变更：

1. CMakeLists.txt 变更：

-find_package(QT NAMES Qt6 Qt5 REQUIRED COMPONENTS Core)
+find_package(QT NAMES Qt6 REQUIRED COMPONENTS Core)

这个变更移除了Qt5的支持，只保留Qt6。这个改动是合理的，因为：

• 代码维护更简单，不需要同时支持两个版本
• Qt6是最新版本，提供更好的性能和特性
• 但需要注意这会提高系统的最低依赖要求

2. InformationDialog.qml 变更：

description: {
    var originalDim = IV.FileControl.slotGetInfo("OriginalDimension", filePath);
    if (originalDim && originalDim !== "-") {
        return originalDim;
    } else {
        return imageInfo.width + "x" + imageInfo.height;
    }
}

这个改动优化了尺寸显示逻辑：

• 优点：优先显示原始尺寸，提供更准确的信息
• 建议：可以添加错误处理，防止slotGetInfo调用失败

3. unionimage.cpp 变更：
   主要改进了图片处理逻辑，增加了大图片支持：

a) 内存限制设置：

reader.setAllocationLimit(2048);

• 优点：防止内存溢出
• 建议：2048MB的限制值可以考虑做成可配置项

b) 图片尺寸缩放逻辑：

const int maxDimension = 4096;
if (originalSize.width() > maxDimension || originalSize.height() > maxDimension) {
    QSize scaledSize = originalSize;
    scaledSize.scale(maxDimension, maxDimension, Qt::KeepAspectRatio);
    reader.setScaledSize(scaledSize);
}

优点：

• 防止超大图片占用过多内存
• 保持图片比例
• 添加了详细的日志记录

建议改进：

1. 错误处理：

if (!reader.canRead()) {
    qCWarning(logImageViewer) << "Cannot read image:" << path;
    return false;
}

2. 资源管理：

// 在读取大图片前检查可用内存
if (originalSize.width() * originalSize.height() * 4 > getAvailableMemory()) {
    qCWarning(logImageViewer) << "Insufficient memory for image loading";
    return false;
}

3. 性能优化：

// 添加图片加载进度反馈
QFuture<QImage> future = QtConcurrent::run([&]() {
    return reader.read();
});

4. 安全性考虑：

// 验证图片文件大小
QFileInfo fileInfo(path);
if (fileInfo.size() > MAX_FILE_SIZE) {
    qCWarning(logImageViewer) << "File too large:" << path;
    return false;
}

总体评价：
这个改动主要解决了大图片处理的问题，增加了内存限制和自动缩放功能。代码质量良好，但可以进一步加强错误处理和安全性检查。建议添加更多的边界条件检查和资源管理机制。

[deepin-ci-robot]

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: add-uos, lzwind

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

[add-uos]

/forcemerge

[deepin-bot]

This pr force merged! (status: unstable)