JSONTokener should implement java.io.Closeable

Aperta

#718 aperta il 28/1/2023 9 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà: 2/5
Tempo stimato: 1-3 ore
Idoneità per principianti: 45/100

Tipo di issue: Funzionalità
Chiarezza: Specificata chiaramente
Stato di attività: Ferma
Stack tecnologico: java
Ambito: backend-api-design

Direzione di ricerca

Start by locating JSONTokener and its Reader and InputStream constructors, then review how those inputs are retained. Done means JSONTokener implements java.io.Closeable and closing it closes the supplied reader or stream while preserving Java 6 compatibility.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

No changes at this time

While JSONTokener(java.io.Reader) and JSONTokener(java.io.InputStream) constructors have explicit notes about JSONTokener not closing the reader/stream, I’d like to have it close the reader/stream. Implementing java.io.Closeable isn’t backward-incompatible (if the library user doesn’t explicitly close it, nothing happens), and for those who use try-with-resources construct, having to keep a separate reference for the reader/stream is a pain. Although try-with-resources itself is Java 7+, the java.io.Closeable interface is @since 1.5, so implementing the interface can be done without losing Java 6 compatibility.

Lingua principale: Java
Stelle: 4.7k
Fork: 2.6k
Merge medio: 6g 20h
PR unite (30g): 2

Guida per i contributori

Apri la guida per i contributori

Come iniziare

Leggi tutta la issue e poi la guida ai contributi del progetto.
Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
Fai un fork del repository e lavora su un branch.
Apri una pull request che faccia riferimento al numero della issue.

Altre issue di stleary/JSON-java

JSONPointer pre-existing bug in toURIFragment() Aperta

Fix before the next release

Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100

stleary/JSON-java#1082 · Aperta il 20/9/2026
New JSONPointer tests needed Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100

stleary/JSON-java#1081 · Aperta il 20/9/2026
JSONML checks childNode instead of the reserved childNodes attribute Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100

stleary/JSON-java#1080 · Aperta il 19/9/2026
JSONML decodes text entities twice with keepStrings enabled Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100

stleary/JSON-java#1079 · Aperta il 19/9/2026
JSONObject.toString does not use an Enum's `toString()` but its `name()` Aperta

Fix before the next release

Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100

stleary/JSON-java#838 · 16 commenti · 3 reazioni · Aperta il 11/12/2023

Tutte le issue di stleary/JSON-java

Issue simili

build-scan-xunit: accept ISO offset timestamps in test reports Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

elastic/gradle-plugins#157 · Aperta il 23/9/2026
Counterexamples differ when using `-recover` (`-workers`, `-fp`, `-seed` must match the original run) Aperta

enhancement Tools

Difficoltà 1/5 Meno di un'ora Idoneità per principianti 75/100

tlaplus/tlaplus#1444 · 2 commenti · Aperta il 23/9/2026
[Studio][Bug] rmq.instance.metrics advertises an invalid MCP input contract Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

apache/rocketmq-dashboard#5008 · Aperta il 23/9/2026
[Bug] A cell read into a ReadCellData field loses its row and column index Aperta

bug

Difficoltà 2/5 1-3 ore Idoneità per principianti 75/100

apache/fesod#1131 · 1 commento · Aperta il 23/9/2026
DETECT_PARAMETER_NAMES=false silently disables @ConstructorProperties-based Creator detection too Aperta

Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100

FasterXML/jackson-databind#6229 · Aperta il 23/9/2026

Altre issue su Java