RFR: JDK-8309550: jdk.jfr.internal.Utils::formatDataAmount method should gracefully handle amounts equal to Long.MIN_VALUE [v2]

Markus Grönlund mgronlun at openjdk.org
Wed Jun 7 11:43:59 UTC 2023


On Wed, 7 Jun 2023 10:07:18 GMT, Frederic Thevenet <fthevenet at openjdk.org> wrote:

>> Please review this simple fix to JDK-8309550.
>> 
>> NB: The problem originally reported only concerned `src/jdk.jfr/share/classes/jdk/jfr/internal/Utils.java`, but since this code was duplicated in `src/jdk.jfr/share/classes/jdk/jfr/internal/util/ValueFormatter.java` with, as far as I understood from the bug description, with an intent to refactor it a a later time, I opted to change both occurrences of the method.
>
> Frederic Thevenet has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Put brackets around the ? condition

Marked as reviewed by mgronlun (Reviewer).

Normally we are ok with only one Reviewer. But I approved as well. Cheers.

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

PR Review: https://git.openjdk.org/jdk/pull/14341#pullrequestreview-1467426126
PR Comment: https://git.openjdk.org/jdk/pull/14341#issuecomment-1580619803


More information about the hotspot-jfr-dev mailing list