RFR: 7903788: preparation towards json configuration for jextract tool
Maurizio Cimadamore
mcimadamore at openjdk.org
Fri Aug 16 13:28:00 UTC 2024
On Fri, 16 Aug 2024 11:49:29 GMT, Athijegannathan Sundararajan <sundar at openjdk.org> wrote:
> * adding json skara code into org.openjdk.jextract.json.parser package. Minor changes done - pretty printing, JSONValue is made sealed and access changes (private final fields).
> * org.openjdk.jextract.json package for JSON interface to the rest of the jextract tool. Main interface is org.openjdk.jextract.json.JSON. parse and toString methods to convert record into JSON String and vice versa.
> * Other classes are public mainly for testing purposes. build.gradle adjusted with the required --add-exports clauses so that jextract module does not have to export the new json packages.
test/testng/org/openjdk/jextract/test/json/parser/JSONParserTests.java line 205:
> 203: @Test
> 204: void testLargerJSONText() {
> 205: var text = "{\n" +
Would it make sense, for this or other big chunks of JSON, to do a roundtrip test where we parse the text, we get a JSONValue, then we obtain the string representation from it, and we compare it against the text?
-------------
PR Review Comment: https://git.openjdk.org/jextract/pull/257#discussion_r1719838826
More information about the jextract-dev
mailing list