> Overall this looks good, but I do have to wonder if it is needed given > that your previous patch already has documentation for each function. Agreed, it looks like too much information. This second patch was created after the review of v1, and to be honest, I think that this overview is more clear as it focus in the what we want (write a JSON) instead of documenting each function. So, if I need to choose one of them, I'll choose this. > But I'll leave that decision to you. Ok, I'll keep the second! > Thanks for working on this! Thank you!
