Skip to content

add xy_streql_ic#242

Merged
ccmywish merged 5 commits into
RubyMetric:devfrom
Mikachu2333:feat/support_upper
Aug 7, 2025
Merged

add xy_streql_ic#242
ccmywish merged 5 commits into
RubyMetric:devfrom
Mikachu2333:feat/support_upper

Conversation

@Mikachu2333

Copy link
Copy Markdown
Collaborator

问题描述

  1. 希望支持大写
  2. fix mirror code 支持大写 #241

方案与实现

  1. xy.h 里面增加了一个函数 xy_streql_ic
  2. 批量替换了原始函数

@github-actions

github-actions Bot commented Aug 7, 2025

Copy link
Copy Markdown

Hi @Mikachu2333,

❤️ 感谢你的贡献!我们将在5天内阅读此 PR 并回复你

Comment thread lib/xy.h
@ccmywish

ccmywish commented Aug 7, 2025

Copy link
Copy Markdown
Contributor

@Mikachu2333 👍

我觉得这个函数是有意义的,但是全部把 xy_streql() 替换成新函数 xy_streql_ic() 是不应该的。

比如:

  1. 命令行选项的大小写有着严格的要求,虽然我们现在用了全称的形式,不像 -x -X 这样只有一个字母,但是大小写依然是有意义的。对此,我暂时持保留意见。

  2. 有一些场景,只会在内部出现,比如 first upstream 这都是内部值,不可能出现大写的情况,用 xy_streql_ic() 相当于放宽了范围,维护者看代码的时候乍一看会不明白为什么会这里需要 ignore case,从而怀疑程序路径有多处。

如果是 mirror code,我认为不区分大小写是有意义的 👍


可以遵从一个原则:如无必要,应当使用 xy_streql() 而非扩大了可接受范围的 xy_streql_ic()

请 revert 第二次 commit,然后仅在真正需要 ignore case 的情况下再使用 xy_streql_ic()

@ccmywish ccmywish added this to the v0.2.3 milestone Aug 7, 2025
@ccmywish ccmywish added the 改善加强 改善加强 label Aug 7, 2025
@Mikachu2333

Copy link
Copy Markdown
Collaborator Author
1. 命令行选项的大小写有着严格的要求,虽然我们现在用了全称的形式,不像 `-x` `-X` 这样只有一个字母,但是大小写依然是有意义的。对此,我暂时持保留意见。

如果是 mirror code,我认为不区分大小写是有意义的 👍

你说得对,我欠考虑了

@Mikachu2333

Copy link
Copy Markdown
Collaborator Author

现在只把main中的内容改了,然后target也设置成了ignore case

@Mikachu2333

Copy link
Copy Markdown
Collaborator Author

考虑了一下,就改target算完

@ccmywish ccmywish merged commit 695ce10 into RubyMetric:dev Aug 7, 2025
1 of 2 checks passed
@Mikachu2333 Mikachu2333 deleted the feat/support_upper branch August 7, 2025 16:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

改善加强 改善加强

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants