Skip to content

docs(net-filter): clarify optional debug argument usage - #148

Open
yuKing123-king wants to merge 1 commit into
DKapture:mainfrom
yuKing123-king:fix/net-filter-debug-help
Open

yuKing123-king wants to merge 1 commit into
DKapture:mainfrom
yuKing123-king:fix/net-filter-debug-help

Conversation

@yuKing123-king

Copy link
Copy Markdown
Contributor

修复 net-filter debug 参数帮助信息不准确的问题。

当前 --debug/-d 使用 optional_argument,按 getopt_long 语义,短选项带值需要写成 -d9,长选项带值需要写成 --debug=9。写成 -d 9 或 --debug 9 时,9 会被当作普通位置参数,optarg 为空,实际不会设置 debug level。

本次修改只更新 help 文案,明确提示:

  • 短选项使用 -d[level],例如 -d9
  • 长选项使用 --debug[=level],例如 --debug=9
  • 不要使用 -d 9 或 --debug 9

这样可以避免用户按错误形式传参后误以为 debug level 已生效.

@github-actions

github-actions Bot commented Sep 14, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

(Review updated until commit c93dd22)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@yuKing123-king
yuKing123-king force-pushed the fix/net-filter-debug-help branch from 05d29e7 to f4d6fef Compare September 14, 2026 06:55
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit f4d6fef

@yuKing123-king
yuKing123-king force-pushed the fix/net-filter-debug-help branch 2 times, most recently from 552f236 to b9d54a7 Compare September 14, 2026 08:26
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit 552f236

@yuKing123-king
yuKing123-king force-pushed the fix/net-filter-debug-help branch from b9d54a7 to c24c9c6 Compare September 14, 2026 08:27
Signed-off-by: Wang Yu <wangyu6@uniontech.com>
@yuKing123-king
yuKing123-king force-pushed the fix/net-filter-debug-help branch from c24c9c6 to c93dd22 Compare September 14, 2026 08:27
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit c24c9c6

@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit c93dd22

1 similar comment
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit c93dd22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant