<Sound Dev> [13] Review Request: 8221436 Incorrect check of package in Line.Info.toString()

Dan Rollo danrollo at gmail.com
Tue Mar 26 18:28:49 UTC 2019


+1, Looks good to me.

OOP for the win!

Dan

> On Mar 25, 2019, at 10:01 PM, Sergey Bylokhov <Sergey.Bylokhov at oracle.com> wrote:
> 
> Hello, Audio Guru.
> 
> Please review the small fix for jdk13.
> 
> Bug: https://bugs.openjdk.java.net/browse/JDK-8221436
> Fix: http://cr.openjdk.java.net/~serb/8221436/webrev.00
> 
> The Line.Info.toString() method tries to cut the package string for the build-in classes.
> But it does not take into account the custom packages like "bug.javax.sound.sampled".
> 
> Since this is the toString() method, I made the fixed code as compact and simple as possible.
> 
> -- 
> Best regards, Sergey.



More information about the sound-dev mailing list