<html>
<head>
<meta content="text/html; charset=UTF-8" http-equiv="Content-Type">
</head>
<body bgcolor="#FFFFFF" text="#000000">
Looks fine.<br>
<br>
<meta http-equiv="content-type" content="text/html; charset=UTF-8">
<pre>// this test just defend from the accidental changes.
should be written
<meta http-equiv="content-type" content="text/html; charset=UTF-8">// this test just defends against accidental changes.
<pre></pre>
</pre>
-phil.<br>
<br>
On 3/25/19, 7:01 PM, Sergey Bylokhov wrote:
<blockquote
cite="mid:a040d63b-a842-639e-628c-c9f2804d5a6c@oracle.com"
type="cite">Hello, Audio Guru.
<br>
<br>
Please review the small fix for jdk13.
<br>
<br>
Bug: <a class="moz-txt-link-freetext" href="https://bugs.openjdk.java.net/browse/JDK-8221436">https://bugs.openjdk.java.net/browse/JDK-8221436</a>
<br>
Fix: <a class="moz-txt-link-freetext" href="http://cr.openjdk.java.net/~serb/8221436/webrev.00">http://cr.openjdk.java.net/~serb/8221436/webrev.00</a>
<br>
<br>
The Line.Info.toString() method tries to cut the package string
for the build-in classes.
<br>
But it does not take into account the custom packages like
"bug.javax.sound.sampled".
<br>
<br>
Since this is the toString() method, I made the fixed code as
compact and simple as possible.
<br>
<br>
</blockquote>
</body>
</html>