Use iterative parse method instead of recursive parse #4

wants to merge 1 commit into


None yet

3 participants

calio commented Jul 2, 2012

The original recursive parse would cause stack size exceeding system limit and a segment fault when dealing with large date ( usually 100k lines or more ).

agentzh commented Jul 3, 2012

I'll look into this. Thanks for the patch!

msva commented Aug 8, 2015

@agentzh, ping?

agentzh commented Aug 10, 2015

@msva I'll look into this at some point. Sorry for the delay.

@cryptofuture cryptofuture added a commit to cryptofuture/rds-json-nginx-module that referenced this pull request May 5, 2016
@cryptofuture cryptofuture apply pull #4 a6c6832
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment