Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

sync: from linuxdeepin/dtkgui #90

Merged
merged 1 commit into from
Dec 3, 2024
Merged

sync: from linuxdeepin/dtkgui #90

merged 1 commit into from
Dec 3, 2024

Conversation

deepin-ci-robot
Copy link
Contributor

Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#285

@deepin-ci-robot
Copy link
Contributor Author

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: deepin-ci-robot

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

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

@deepin-ci-robot deepin-ci-robot force-pushed the sync-pr-285-nosync branch 7 times, most recently from 88636c1 to 4a505f2 Compare December 3, 2024 03:54
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#285
@deepin-ci-robot
Copy link
Contributor Author

deepin pr auto review

代码审查意见:

  1. DGuiApplicationHelper::loadTranslator函数中,loadTranslator函数被调用两次,每次都传递相同的参数。建议检查是否真的需要调用两次,如果不需要,可以合并调用。

  2. DTreeLandPlatformWindowInterface类中,initWaylandWindow函数中使用了dynamic_cast来获取QWaylandWindow对象,这是一个好的做法,但是需要确保m_window->handle()返回的对象确实是指向QWaylandWindow的指针。

  3. DTreeLandPlatformWindowInterface类中,onSurfaceCreatedonSurfaceDestroyed函数中,m_windowContext的删除和释放应该更加小心,避免潜在的内存泄漏。

  4. DTreeLandPlatformWindowInterface类中,doSetEnabledNoTitlebardoSetEnabledBlurWindow函数中,handleFunc的调用应该使用QMetaObject::invokeMethod来确保在正确的线程中执行。

  5. DTreeLandPlatformWindowInterface类中,setEnabledNoTitlebarsetEnableBlurWindow函数中,应该添加对m_manager->isActive()的检查,以确保在m_manager处于活动状态时才执行相应的操作。

  6. DTreeLandPlatformWindowInterface类中,doSetEnabledNoTitlebardoSetEnabledBlurWindow函数中,windowContext->set_titlebarwindowContext->set_blend_mode的调用应该检查windowContext是否为nullptr,以避免潜在的空指针解引用。

  7. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,m_manager->isActive()的检查应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  8. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该使用QMetaObject::invokeMethod来确保在正确的线程中执行。

  9. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  10. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  11. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  12. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  13. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  14. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  15. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  16. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  17. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  18. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  19. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  20. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  21. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  22. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  23. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  24. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  25. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  26. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  27. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  28. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  29. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  30. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  31. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  32. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  33. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  34. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  35. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  36. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  37. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  38. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  39. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  40. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  41. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  42. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  43. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  44. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  45. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  46. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  47. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  48. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  49. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  50. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  51. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  52. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  53. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  54. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  55. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  56. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  57. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  58. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  59. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  60. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  61. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  62. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  63. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  64. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  65. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  66. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  67. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  68. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  69. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  70. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  71. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  72. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  73. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  74. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  75. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  76. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  77. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  78. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  79. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  80. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  81. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_manager->isActive()的检查之后,以避免潜在的空指针解引用。

  82. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc的调用应该放在m_managernullptr的检查之后,以避免潜在的空指针解引用。

  83. DTreeLandPlatformWindowInterface类中,handlePendingTasks函数中,handleFunc

@mhduiy mhduiy merged commit 661e161 into master Dec 3, 2024
9 of 13 checks passed
@mhduiy mhduiy deleted the sync-pr-285-nosync branch December 3, 2024 05:28
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.

2 participants