Integrated: 8263659: Reflow GTestResultParser for better readability

Alex Blewitt github.com+76791+alblue at openjdk.java.net
Thu Mar 18 16:44:42 UTC 2021


On Sat, 13 Mar 2021 22:13:39 GMT, Alex Blewitt <github.com+76791+alblue at openjdk.org> wrote:

> As noted by https://sonarcloud.io/code?id=shipilev_jdk&selected=shipilev_jdk%3Atest%2Fhotspot%2Fjtreg%2Fgtest%2FGTestResultParser.java there are a few fixes that can be applied for the GTestResultParser:
> 
> * Avoiding nested 'try' statements
> * Avoiding nested 'switch' statements
> * Adding a break for each switch case to prevent accidental/unwanted fall-through
> * Disabling ability to load from remote files when parsing XML files

This pull request has now been integrated.

Changeset: 21db0f67
Author:    Alex Blewitt <alex.blewitt at gmail.com>
Committer: Igor Ignatyev <iignatyev at openjdk.org>
URL:       https://git.openjdk.java.net/jdk/commit/21db0f67
Stats:     25 lines in 1 file changed: 3 ins; 4 del; 18 mod

8263659: Reflow GTestResultParser for better readability

Reviewed-by: shade, iignatyev

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

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


More information about the hotspot-dev mailing list