Tencent / TencentKona-17

Tencent Kona JDK17 is a no-cost, production-ready distribution of the Open Java Development Kit (OpenJDK), Long-Term Support(LTS) with quarterly updates. Tencent Kona JDK17 is certified as compatible with the Java SE standard.

Other

156 stars 32 forks source link

Remove redundant spaces and lines #32

Open mck156 opened 4 months ago

mck156 commented 4 months ago

20

tencent-adm commented 4 months ago

All committers have signed the CLA.

johnshajiang commented 4 months ago

我以为，那些被删除的空行是需要的。它们会让代码的结构更清晰。

mck156 commented 4 months ago

@johnshajiang 那我恢复全部的代码空行，但是我想把注释的空行删减保留，我感觉那个是可以的。

johnshajiang commented 4 months ago

但是我想把注释的空行删减保留，我感觉那个是可以的。

在OpenJDK的其它文件中也是用的双空格，比如[Thread.java]中的版本声明。其实有很多注释也是这么用的。关于这个问题，可能需要了解一点儿[历史背景]。

[Thread.java]: https://github.com/Tencent/TencentKona-17/blob/master/src/java.base/share/classes/java/lang/Thread.java#L7

[历史背景]: https://www.quora.com/Why-do-some-people-still-use-two-spaces-following-a-period

mck156 commented 4 months ago

但是我想把注释的空行删减保留，我感觉那个是可以的。

在OpenJDK的其它文件中也是用的双空格，比如 Thread.java 中的版本声明。其实有很多注释也是这么用的。关于这个问题，可能需要了解一点儿历史背景。

了解！那我把它更改回来。

mck156 commented 4 months ago

@johnshajiang 我已经恢复了所有的空行以及那一处双空格！

johnshajiang commented 4 months ago

@mck156 你是否使用脚本或程序去查看了所有的腾讯文件，以确保此类问题都被找到了？如果有的话，请将脚本或程序复制到这里。

mck156 commented 4 months ago

@johnshajiang 我只用了“XX”进行全局搜索，还加了 in:file进行搜索也只搜索到这个，有些没搜索到的可能是因为被修改了。

mck156 commented 4 months ago

@johnshajiang 另外，我不加任何条件的搜索也很多在TencentKona-8里，基本上没有TencentKona-17的内容，所以就提交了！

mck156 commented 4 months ago

PixPin_2024-07-11_10-07-52 这是我的搜索结果！

mck156 commented 4 months ago

@johnshajiang 我看了下我的搜索结果和脚本内容一致，所以直接提交这个结果供您参考！

johnshajiang commented 4 months ago

我已经在研学基地中将该issue的状态标注为已完成，请查看。但这个PR目前还不会被合并。

mck156 commented 4 months ago

@johnshajiang 查找字段逗号的问题，我也看了#29 的建议，这是加上逗号的搜索结果！

mck156 commented 4 months ago