Skip to content

Conversation

@myk1343
Copy link
Contributor

@myk1343 myk1343 commented Apr 1, 2025

修复查看750 X 7882尺寸图片问题

Bug: https://pms.uniontech.com/bug-view-310713.html
Log: 修复查看750 X 7882尺寸图片问题

修复查看750 X 7882尺寸图片问题

Bug: https://pms.uniontech.com/bug-view-310713.html
Log: 修复查看750 X 7882尺寸图片问题
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • if (!reader.canRead())条件分支中,重新设置了reader.setFileName(path),这可能是多余的,因为如果reader.canRead()返回false,说明文件无法读取,重新设置文件名不会改变这一事实。
  • reader.setDecideFormatFromContent(true)reader.setAutoDetectImageFormat(true)的设置可能会影响性能,因为它们会尝试从文件内容中推断格式,而不是依赖于文件扩展名。

是否建议立即修改:

  • 是,建议立即修改以避免潜在的性能问题,并确保代码逻辑清晰。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: max-lvs, myk1343

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@myk1343
Copy link
Contributor Author

myk1343 commented Apr 1, 2025

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented Apr 1, 2025

This pr force merged! (status: unstable)

@deepin-bot deepin-bot bot merged commit 113ebd0 into linuxdeepin:develop/eagle Apr 1, 2025
18 of 20 checks passed
@myk1343 myk1343 deleted the uploadcode branch April 14, 2025 08:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants