Skip to content

Conversation

@robertkill
Copy link
Contributor

as title

log: as title

@BLumia BLumia force-pushed the master branch 3 times, most recently from 12fc117 to 5a48931 Compare March 4, 2025 07:18
@robertkill robertkill force-pushed the master branch 2 times, most recently from 5b3178d to d792011 Compare March 4, 2025 09:42
@deepin-bot
Copy link
Contributor

deepin-bot bot commented Mar 6, 2025

TAG Bot

New tag: 2.0.6
DISTRIBUTION: unstable
Suggest: synchronizing this PR through rebase #33

as title

log: as title
pms: TASK-374223
@robertkill robertkill force-pushed the master branch 8 times, most recently from 9815304 to 2619c53 Compare March 20, 2025 05:56
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. debian/rules文件中,移除了QT_SELECT=5环境变量,这可能会影响Qt5和Qt6之间的兼容性。建议确认是否有必要移除该变量,或者是否需要根据实际情况设置不同的Qt版本。

  2. drivermanger.cppworkmodule.cpp文件中,QCameraQCameraImageCapture类被替换为QMediaDevicesQImageCapture。这可能是为了使用Qt6的新API。但是,需要确保这些更改不会破坏现有的功能,并且所有相关的代码都已经更新以适应这些API的变化。

  3. ErollThreadVerifyThread类中,updateCameraState函数被注释掉了。如果这个函数不再需要,应该完全移除它。否则,应该检查是否有其他地方调用了这个函数,并确保它仍然被需要。

  4. ErollThreadVerifyThread类的captureError函数中,错误处理逻辑被修改了。现在,只有当错误码大于0时才会发出错误信号。这可能是为了更精确地处理错误。但是,应该确保这个更改不会导致任何意外的行为。

  5. workmodule.h文件中,QCameraQCameraImageCapture类被替换为QCameraQImageCapture。这可能是为了使用Qt6的新API。但是,需要确保这些更改不会破坏现有的功能,并且所有相关的代码都已经更新以适应这些API的变化。

  6. ErollThreadVerifyThread类的Stop函数中,m_imageCapture->cancelCapture()被移除了。如果这个函数不再需要,应该完全移除它。否则,应该检查是否有其他地方调用了这个函数,并确保它仍然被需要。

  7. ErollThreadVerifyThread类的Stop函数中,disconnect被用来断开m_imageCapture的信号和槽。这可能是为了防止在相机停止后继续处理捕获的图像。但是,应该确保这个更改不会导致任何意外的行为。

  8. ErollThreadVerifyThread类的Stop函数中,m_camera->unload()被移除了。如果这个函数不再需要,应该完全移除它。否则,应该检查是否有其他地方调用了这个函数,并确保它仍然被需要。

  9. ErollThreadVerifyThread类的Stop函数中,for循环被用来释放m_charaDatas中的每个元素。这可能是为了防止内存泄漏。但是,应该确保这个更改不会导致任何意外的行为。

  10. ErollThreadVerifyThread类的Stop函数中,close(m_fileSocket)被调用。这可能是为了关闭文件描述符。但是,应该确保这个更改不会导致任何意外的行为。

总体来说,这些更改看起来是为了适应Qt6的新API,并且大多数更改都是为了提高代码的健壮性和性能。但是,需要确保这些更改不会破坏现有的功能,并且所有相关的代码都已经更新以适应这些API的变化。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: BLumia, ECQZXC, robertkill, yixinshark

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

@robertkill robertkill merged commit e3b21c1 into linuxdeepin:master Mar 20, 2025
9 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.

5 participants