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

optimize: packet builder & string/bytes I/O process #32

Merged
merged 8 commits into from
May 7, 2024
Merged

optimize: packet builder & string/bytes I/O process #32

merged 8 commits into from
May 7, 2024

Conversation

fumiama
Copy link
Member

@fumiama fumiama commented May 6, 2024

  • 注意到utils.PacketBuilderbinary.Builder极为相似,因此将二者合并(有助于日后池化)。
  • 优化string[]bytes互转为零拷贝的utils.B2Sutils.S2B(为引入此方法,升级go版本到1.20)。
  • main.go自动生成的文件添加到ignore。
  • 移除main.go中包含的包括qq号在内的敏感信息。
  • 其它代码结构优化,包括
    • 替换无用的make([]byte, 0)为等价的nil
    • Pack的默认参数-1添加宏定义binary.PackTypeNone
    • 移除info/sig.go中默认空值的冗余声明使其更简洁。
    • NewBuilder添加64字节的预分配空间以减少append时的扩容次数。

@Redmomn Redmomn merged commit bdec9a4 into LagrangeDev:master May 7, 2024
2 checks passed
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.

2 participants