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.

Sign up for GitHub

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

Jump to bottom

Replace bytes.Buffer with strings.Builder #94

Merged

zhengchun merged 2 commits into antchfx:master from egginabucket:master

Dec 8, 2022

Contributor

egginabucket commented Dec 6, 2022

strings.Builder has significantly better performance than bytes.Buffer, and should be used instead.

For tests, strings.Index(s, substr) == -1 has also been replaced with !strings.Contains(s, substr)

egginabucket added 2 commits

December 6, 2022 09:43


          replace strings.Index == -1 with Contains

59104e5


          replace bytes.Buffer with strings.Builder

b9c0fac

zhengchun merged commit 9dbfa11 into antchfx:master

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment