-
Notifications
You must be signed in to change notification settings - Fork 8k
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
why use github.com/json-iterator/go #1008
Comments
We have integrated it because it's faster than the stdlib |
sorry about the bug, I will fix it as soon as possible. Test coverage is our recent focus. |
should be fixed now, you need to use the latest master branch. The map key is read from the buffer without copy, which caused this bug. |
fix from json-iterator/go@bd4364a Thanks @taowen |
awesome, thanks @taowen 😄 |
@javierprovecho @appleboy thanks for choosing jsoniter. There are bugs for sure, I can only promise to fix them as soon as I can. |
jsoniter also does not seem to work on Google App Engine (use of unsafe). One option may be to enable/disable this via build tags. |
jsoniter is based on unsafe.Pointer which seems not allowed by GAE. build tags is an option. |
Wow. Nice work with jumping on this so quick. I think refactoring to enable via a built-tag is the right idea. It allows evolution and innovation, while reducing the risk by giving it time to bake. |
See #1026 |
i have occurred an error in using json-iterator
json-iterator/go#129
The text was updated successfully, but these errors were encountered: