RFR: 8287008: Improve tests for thread dumps in JSON format [v4]

Alan Bateman alanb at openjdk.java.net
Mon May 23 16:58:43 UTC 2022


On Mon, 23 May 2022 16:44:12 GMT, Chris Plummer <cjplummer at openjdk.org> wrote:

>> Alan Bateman has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Use string rater than number for process/thread identifiers
>
> test/lib/jdk/test/lib/threaddump/ThreadDump.java line 85:
> 
>> 83:  *         "threads": [...],
>> 84:  *         "threadCount": "1"
>> 85:  *       }
> 
> Why is there no longer a ',' here?

I had to collapse several objects and reduce the number of items in the arrays in order to get it down to an example that fits into the class description. To yes, all items except the last should have a trailing comma.

-------------

PR: https://git.openjdk.java.net/jdk/pull/8784


More information about the serviceability-dev mailing list