Skip to content

refactor(excel): remove unused biffh header file#55

Merged
deepin-bot[bot] merged 1 commit intolinuxdeepin:develop/20260424from
pppanghu77:dev
Apr 28, 2026
Merged

refactor(excel): remove unused biffh header file#55
deepin-bot[bot] merged 1 commit intolinuxdeepin:develop/20260424from
pppanghu77:dev

Conversation

@pppanghu77
Copy link
Copy Markdown
Contributor

Remove the biffh.hpp header file which is no longer used in the Excel parsing module. This file contained BIFF record definitions and was likely part of an older implementation approach. Log: refactor(excel): remove unused biffh header file Task: https://pms.uniontech.com/task-view-388905.html

Remove the biffh.hpp header file which is no longer used in the
Excel parsing module. This file contained BIFF record definitions
and was likely part of an older implementation approach.
Log: refactor(excel): remove unused biffh header file
Task: https://pms.uniontech.com/task-view-388905.html
@deepin-ci-robot
Copy link
Copy Markdown

deepin pr auto review

@github-actions
Copy link
Copy Markdown

  • 敏感词检查失败, 检测到1个文件存在敏感词
详情
{
    "3rdparty/libs/fileext/excel/xlsxio/xlsxio_read.c": [
        {
            "line": "        if (XML_Char_icmp(reltype, X(\"http://schemas.openxmlformats.org/officeDocument/2006/relationships/worksheet\")) == 0) {",
            "line_number": 845,
            "rule": "S35",
            "reason": "Url link | 591ded0820"
        },
        {
            "line": "        } else if (XML_Char_icmp(reltype, X(\"http://schemas.openxmlformats.org/officeDocument/2006/relationships/sharedStrings\")) == 0) {",
            "line_number": 853,
            "rule": "S35",
            "reason": "Url link | b67c93b15a"
        },
        {
            "line": "        } else if (XML_Char_icmp(reltype, X(\"http://schemas.openxmlformats.org/officeDocument/2006/relationships/styles\")) == 0) {",
            "line_number": 858,
            "rule": "S35",
            "reason": "Url link | 7b6758f750"
        }
    ]
}

@deepin-ci-robot
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by: Johnson-zs, max-lvs, pppanghu77

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

@pppanghu77
Copy link
Copy Markdown
Contributor Author

/forcemerge

@deepin-bot
Copy link
Copy Markdown
Contributor

deepin-bot Bot commented Apr 28, 2026

This pr force merged! (status: unstable)

@deepin-bot deepin-bot Bot merged commit 5bc3e67 into linuxdeepin:develop/20260424 Apr 28, 2026
18 of 21 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.

4 participants