Chuyu-Team / YY-Thunks

Fix DecodePointer, EncodePointer,RegDeleteKeyEx etc. APIs not found in Windows XP RTM.
MIT License
567 stars 103 forks source link

Convert to use namespace YY::Thunks; namespace YY::Thunks::internal; namespace YY::Thunks::Fallback #84

Closed lygstate closed 4 months ago

lygstate commented 4 months ago

同事替换tab 为4space 删除结尾空格 删除冗余缩进

mingkuang-Chuyu commented 4 months ago

非常感谢提交PR,但是我个人没有仅仅为了调整格式而引入大范围的更改习惯。

不过未来,当某文件需要更新时,我会统一更新它。

再次感谢。

lygstate commented 4 months ago

非常感谢提交PR,但是我个人没有仅仅为了调整格式而引入大范围的更改习惯。

不过未来,当某文件需要更新时,我会统一更新它。

再次感谢。

我想合并这个,这样方便我合并一些其它改动

mingkuang-Chuyu commented 4 months ago

@lygstate 我希望只提交一些必要更改就可以了。因为现在的改动点实在是太多了……

lygstate commented 4 months ago

@lygstate 我希望只提交一些必要更改就可以了。因为现在的改动点实在是太多了……

或者你可以自己提交一下, 先全局替换 tab 为4个空格 这个我很需要, 还有就是把行尾空格都删掉 这样就不需要review 了

mingkuang-Chuyu commented 4 months ago

@lygstate 伙计,这不是review的问题,而是产生了很多没有实质性影响的更改。我需要避免的是这个,而并非大量review的问题。

毕竟多一个回车空格还是Tab什么的对程序逻辑没有影响,甚至他们本身因为是不见得的字符,对视觉也没有啥影响(当然某些字体下可能产生一些排版不一致)。

lygstate commented 4 months ago

@lygstate 伙计,这不是review的问题,而是产生了很多没有实质性影响的更改。我需要避免的是这个,而并非大量review的问题。

毕竟多一个回车空格还是Tab什么的对程序逻辑没有影响,甚至他们本身因为是不见得的字符,对视觉也没有啥影响(当然某些字体下可能产生一些排版不一致)。

这个跟开发相关,编辑的时候自动就变成空格了,导致后续改动差异很大

lygstate commented 4 months ago

@lygstate 伙计,这不是review的问题,而是产生了很多没有实质性影响的更改。我需要避免的是这个,而并非大量review的问题。

毕竟多一个回车空格还是Tab什么的对程序逻辑没有影响,甚至他们本身因为是不见得的字符,对视觉也没有啥影响(当然某些字体下可能产生一些排版不一致)。

举个例子,这个里面 https://github.com/Chuyu-Team/YY-Thunks/commit/0bce0d853102a1e209e51f4b2df0facd560b2ff3

空格改动与功能改动混在一起,导致看记录的时候,很难把功能改动与空格改动区分开来, 所以最好的办法是把空格改动在一个版本统一更新了

本身目前就有 .editorconfig 这是很有必要的

mingkuang-Chuyu commented 4 months ago

这些排版什么的不会产生较大负面影响,另外看记录时是可以很容易区分的,因为可以查看差异时可以点击忽略空白字符。

我不接受单纯为了排版而大面积更新代码,因为它没有实质性的影响。

另外项目并没有开启自动代码格式化,您在修改YY-Thunks代码时关闭自动格式化即可。

如果您需要提交功能更改,然后再将相关受影响的文件进行统一格式化。如果不更改,那么暂时无需改动。很抱歉我也没有这方面的强迫症。