Skip to content

Commit

Permalink
Merge pull request #733 from haribabu-dev/jsontokener_closable
Browse files Browse the repository at this point in the history
JSONTokener implemented java.io.Closeable
  • Loading branch information
stleary authored Mar 12, 2023
2 parents 45bcba5 + 7eca507 commit fe22b24
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 6 deletions.
13 changes: 7 additions & 6 deletions src/main/java/org/json/JSONTokener.java
Original file line number Diff line number Diff line change
@@ -1,11 +1,6 @@
package org.json;

import java.io.BufferedReader;
import java.io.IOException;
import java.io.InputStream;
import java.io.InputStreamReader;
import java.io.Reader;
import java.io.StringReader;
import java.io.*;

/*
Public Domain.
Expand Down Expand Up @@ -522,4 +517,10 @@ public String toString() {
return " at " + this.index + " [character " + this.character + " line " +
this.line + "]";
}

public void close() throws IOException {
if(reader!=null){
reader.close();
}
}
}
12 changes: 12 additions & 0 deletions src/test/java/org/json/junit/JSONTokenerTest.java
Original file line number Diff line number Diff line change
Expand Up @@ -313,4 +313,16 @@ public void testNextBackComboWithNewLines() {
assertEquals(0, t2.next());
assertFalse(t2.more());
}

@Test
public void testAutoClose(){
Reader reader = new StringReader("some test string");
try {
JSONTokener tokener = new JSONTokener(reader);
tokener.close();
tokener.next();
} catch (Exception exception){
assertEquals("Stream closed", exception.getMessage());
}
}
}

0 comments on commit fe22b24

Please sign in to comment.