Skip to content

fix: add missing header#319

Merged
ComixHe merged 1 commit intolinuxdeepin:masterfrom
ComixHe:master
Jun 6, 2025
Merged

fix: add missing header#319
ComixHe merged 1 commit intolinuxdeepin:masterfrom
ComixHe:master

Conversation

@ComixHe
Copy link
Copy Markdown
Contributor

@ComixHe ComixHe commented Jun 5, 2025

No description provided.

Signed-off-by: ComixHe <heyuming@deepin.org>
@ComixHe ComixHe requested a review from 18202781743 June 5, 2025 01:54
@deepin-ci-robot
Copy link
Copy Markdown
Contributor

deepin pr auto review

关键摘要:

  • 在新添加的#include <pwd.h>头文件中,需要确认是否有对应的函数调用,以避免编译警告或错误。
  • 如果<pwd.h>头文件的使用是必要的,应该检查是否有对应的错误处理机制,例如处理getpwuid函数可能返回的NULL指针。

是否建议立即修改:

  • 是,需要确保新添加的头文件是必要的,并且在使用相关函数时进行适当的错误处理。

deepin-ci-robot added a commit to linuxdeepin/dtk6gui that referenced this pull request Jun 5, 2025
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#319
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Jun 5, 2025

  • 敏感词检查失败, 检测到1个文件存在敏感词
详情
{
    "src/kernel/dguiapplicationhelper.cpp": [
        {
            "line": "    QString socket_key = \"_d_dtk_single_instance_\";",
            "line_number": 1469,
            "rule": "S106",
            "reason": "Var naming | 2ad926d35b"
        }
    ]
}

@deepin-ci-robot
Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: 18202781743, ComixHe

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

@ComixHe ComixHe merged commit 3f1cfd6 into linuxdeepin:master Jun 6, 2025
22 of 24 checks passed
@18202781743
Copy link
Copy Markdown
Contributor

@ComixHe 下次qt6的pr也合入一下,

18202781743 pushed a commit to linuxdeepin/dtk6gui that referenced this pull request Jun 6, 2025
Synchronize source files from linuxdeepin/dtkgui.

Source-pull-request: linuxdeepin/dtkgui#319
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