fixed warning: initialized field overwritten #6

Closed
wants to merge 1 commit into
from

Projects

None yet

2 participants

@alexly
alexly commented Apr 21, 2012

No description provided.

Owner

I was tempted to do this for v2 but figured out how to suppress the warnings, it was just easier to read/understand by overriding the assignments. Is it ok of I close this pull req since v2 has diverged pretty far, or do you have any other suggestions/tips?

alexly commented Oct 21, 2014

Close it please. I use js0n in the nginx module, but it does not allow warnings during compilation. If I have the time and desire use v2, I will make another pull request. Thanks for your work, it's really great and very fast library.

Owner

Hopefully v2 works out of the box w/o generating any warnings, if it does let me know what they are and what version/compiler is complaining and maybe I can fix it too.

JSON parsing inside nginx is interesting, glad this lib was helpful, thanks!

@quartzjer quartzjer closed this Oct 21, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment