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

我发现的一些笔误和希望提的一些建议 #1

Open
MouriNaruto opened this issue Oct 9, 2023 · 4 comments
Open

我发现的一些笔误和希望提的一些建议 #1

MouriNaruto opened this issue Oct 9, 2023 · 4 comments

Comments

@MouriNaruto
Copy link
Member

MouriNaruto commented Oct 9, 2023

以下是我发现的一些笔误列表(错误描述 -> 正确描述)

  • weakup -> wakeup (在多线程相关设施中)
  • 美剧 -> 枚举 (在 YY C++ 规范文档中)

以下是我希望提的一些建议

  • 论 API 定义那边,我感觉是否应该按照 YY C++ 风味,只在头文件里提供一种接口定义,然后平台细节实现捣鼓到对应的源文件里面,这样也许能在提供统一的 ABI、平台抽象层和未来的平台适配方面应该可能会有些帮助
  • 既然类和结构名已经要求使用帕斯卡命名方式,那么基础设施定义的相关基本类型感觉也遵循这个会更好(譬如 YY::UInt64 会比 YY::uint64_t 看起来要自然不少)
  • 感觉目前的纯头文件实现有些多,感觉 MegaUI 和 YY::Base 应该不是 header-only 库,那么感觉应该要好好利用头文件和源文件分离这件事情,来减少对用户暴露不必要的实现细节

毛利

@LiuPeiqiCN
Copy link

这个项目大概什么时候能够推送?我只能看到一个License文件。

@killaragorn
Copy link

+1

2 similar comments
@engun
Copy link

engun commented Mar 6, 2024

+1

@luutry123
Copy link

+1

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

No branches or pull requests

5 participants