Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Style: 调整代码排版 #148

Merged
merged 9 commits into from
Feb 29, 2024
Merged

Style: 调整代码排版 #148

merged 9 commits into from
Feb 29, 2024

Conversation

yixy-only
Copy link
Collaborator

No description provided.

@yixy-only yixy-only requested a review from wysaid February 22, 2024 10:23
@wysaid
Copy link
Owner

wysaid commented Feb 23, 2024

这是用的 clang-format 吗?

@wysaid
Copy link
Owner

wysaid commented Feb 23, 2024

有一个推荐的风格, 也是目前一直在用的风格, 就是这个:

clang-format.txt

@wysaid
Copy link
Owner

wysaid commented Feb 23, 2024

有一个推荐的风格, 也是目前一直在用的风格, 就是这个:

clang-format.txt

可以考虑把这个放到项目根目录, 然后直接用这个版本的 clang-format 的格式化, 跟你格式化出来的还比较像

@yixy-only
Copy link
Collaborator Author

可以考虑把这个放到项目根目录, 然后直接用这个版本的 clang-format 的格式化, 跟你格式化出来的还比较像

那我先试一下这个 clang-format.txt, 看看有没有什么要细微调整的

@yixy-only
Copy link
Collaborator Author

我觉得可以考虑加一个 .editorconfig 文件来统一行尾序列、字符集、空格代替 Tab, 缩进大小之类的基础编辑配置,这个目前在大部分的 IDE 都支持 @wysaid

@yixy-only yixy-only changed the title Style: 调整代码排版(主要是函数声明) Style: 调整代码排版 Feb 26, 2024
@wysaid
Copy link
Owner

wysaid commented Feb 27, 2024

看着没问题, 要不直接合?

@wysaid
Copy link
Owner

wysaid commented Feb 27, 2024

我这边看着没啥毛病.

@yixy-only
Copy link
Collaborator Author

看着没问题, 要不直接合?

那就直接合吧,也就改代码排版而已,如果后面觉得哪里需要调整的可以再弄 @wysaid

@yixy-only yixy-only merged commit 3c1682a into wysaid:master Feb 29, 2024
1 check passed
@yixy-only yixy-only self-assigned this Apr 25, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants