@haribabu-dev Why should JSONTokener implement Closeable? It is not a closeable resource. Can you just add the close() method without the override or interface?

Ok @stleary , I will do that.

What problem does this code solve?
Users may want to close the reader associated with a JSONTokener

Risks
Low

Changes to the API?
A new public API function was added, with unit test

Will this require a new release?
No

Should the documentation be updated?
No

Does it break the unit tests?
No

Was any code refactored in this commit?
No

Review status
APPROVED

Starting 3-day comment window

@haribabu-dev Why should JSONTokener implement Closeable? It is not a closeable resource. Can you just add the close() method without the override or interface?

Because it (potentially) uses closable resources. See #718, which would have been closed by the original version of this PR, but not the one that got merged.