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