Skip to content

Conversation

@myk1343
Copy link
Contributor

@myk1343 myk1343 commented May 9, 2025

修复右键打印不识别改后缀的图片问题

Bug: https://pms.uniontech.com/bug-view-315823.html
Log: 修复右键打印不识别改后缀的图片问题

修复右键打印不识别改后缀的图片问题

Bug: https://pms.uniontech.com/bug-view-315823.html
Log: 修复右键打印不识别改后缀的图片问题
@deepin-ci-robot
Copy link

deepin pr auto review

关键摘要:

  • reader.jumpToImage(imagePtr->frame);之后添加了对reader.canRead()的检查,确保图像能够被读取。
  • 添加了对图像格式自动检测和内容识别格式的设置,以处理可能出现的格式识别问题。

是否建议立即修改:

  • 否,提交的代码已经实现了对图像加载失败的处理,并且通过自动检测和内容识别格式来提高图像加载的鲁棒性。这些改动是合理的,没有发现需要立即修改的问题。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: myk1343, pengfeixx

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 May 9, 2025

/forcemerge

@deepin-bot
Copy link
Contributor

deepin-bot bot commented May 9, 2025

This pr force merged! (status: unstable)

@deepin-bot deepin-bot bot merged commit e92de82 into linuxdeepin:develop/eagle May 9, 2025
18 of 20 checks passed
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