Skip to content

Conversation

@re2zero
Copy link
Contributor

@re2zero re2zero commented Mar 25, 2025

Enable Qt6 build only for special arch, compat with qt5 build.

Log: Merge Qt5 and Qt6 build.

@re2zero re2zero force-pushed the bugfix branch 2 times, most recently from 5f03250 to 0db83a3 Compare March 25, 2025 05:33
Enable Qt6 build only for special arch, compat with qt5 build.

Log: Merge Qt5 and Qt6 build.
@deepin-ci-robot
Copy link

deepin pr auto review

代码审查意见:

  1. .project 文件被删除,如果这是有意为之,请确保相关的配置信息在其他地方有相应的替代。如果没有,可能会导致项目配置丢失。

  2. dpdfglobal.h 文件中,根据 Qt 版本选择不同的 QMutexLocker 实现方式,这是一个好的做法,可以确保代码的兼容性。

  3. dpdfglobal.cpp 文件中,使用 Q_GLOBAL_STATIC_WITH_ARGS 宏定义了一个递归互斥锁,这是一个好的做法,可以确保线程安全。

  4. debian/control 文件中,Build-Depends 部分对依赖项进行了详细的分类,这有助于确保在不同架构上构建时的兼容性。

  5. debian/rules 文件中,添加了一个检测当前安装的 Qt 版本的宏定义,这是一个好的做法,可以确保使用正确的 Qt 版本进行构建。

  6. reader/document/Model.cpp 文件中,对 unzipCommandpandocCommandhtmltopdfCommand 的执行进行了条件编译,这是一个好的做法,可以确保在不同版本的 Qt 下代码的兼容性。

  7. reader/document/Model.cpp 文件中,decompressor.start(unzipCommand);converter.start(pandocCommand); 以及 converter2.start(htmltopdfCommand); 的调用没有检查命令是否成功启动,可能会导致程序在命令启动失败时继续执行,建议添加相应的错误处理逻辑。

  8. reader/document/Model.cpp 文件中,qDebug()qInfo() 的使用应该根据日志级别进行条件编译,以避免在生产环境中输出过多的调试信息。

  9. reader/document/Model.cpp 文件中,qDebug()qInfo() 的输出应该包含更多的上下文信息,以便于问题的追踪和调试。

  10. reader/document/Model.cpp 文件中,qDebug()qInfo() 的输出应该使用 QStringarg() 方法进行格式化,以提高代码的可读性和可维护性。

以上是本次代码审查的主要意见,希望能够对您有所帮助。

@deepin-ci-robot
Copy link

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: lzwind, re2zero

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

@re2zero
Copy link
Contributor Author

re2zero commented Mar 25, 2025

/merge

@deepin-bot deepin-bot bot merged commit 1a0c4bc into linuxdeepin:master Mar 25, 2025
6 checks passed
@re2zero re2zero deleted the bugfix branch March 25, 2025 09:07
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