Skip to content

Conversation

@Mikachu2333
Copy link
Collaborator

问题描述

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

方案与实现

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

@github-actions
Copy link

github-actions bot commented Aug 7, 2025

Hi @Mikachu2333,

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

@ccmywish
Copy link
Contributor

ccmywish commented Aug 7, 2025

@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
Collaborator Author

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

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

你说得对,我欠考虑了

@Mikachu2333
Copy link
Collaborator Author

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

@Mikachu2333
Copy link
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