Added clear() methods to JSONObject and JSONArray - #577
Conversation
stleary
commented
Dec 4, 2020
@stranck Thanks for submitting this PR. The submitter is usually asked to provide a use case or justification, but I guess in this case it is self-evident. No objection to proceeding after you add the appropriate unit tests. |
(I hope they works :c)
stranck
commented
Dec 4, 2020
I should've added them :) |
stleary
commented
Dec 5, 2020
What problem does this code solve? Risks Changes to the API? Will this require a new release? Should the documentation be updated? Does it break the unit tests? Was any code refactored in this commit? Review status |
Starting 3-day comment window |
I noticed that there wasn't this easy way to clear all the childs of a JSONObject or a JSONArray, so I added it.
Changes in code are really simple