<div dir="ltr">Hello,<div><br></div><div>the backport of the fix for <a href="https://bugs.openjdk.java.net/browse/JDK-8136763">https://bugs.openjdk.java.net/browse/JDK-8136763</a> looks incorrect.</div><div>The corresponding manual test case fails on jdk8.</div><div><br></div><div>The problem: when copy-pasting (or drag-and-dropping) multiple files, the data in the transferable for the flavor "text/uri-list" concats the first two paths.</div><div><br></div><div>If you for example copy:</div><div>/Users/robin/Desktop/file1.txt</div><div>/Users/robin/Desktop/file2.txt<br></div><div>/Users/robin/Desktop/file3.txt<br></div><div><br></div><div>the data in the transferable is</div><div><br></div><div>/Users/robin/Desktop/file1.txt/Users/robin/Desktop/file2.txt<br></div><div>/Users/robin/Desktop/file3.txt<br></div><div><br></div><div>while the expected data is</div><div><br></div><div><div>/Users/robin/Desktop/file1.txt</div><div>/Users/robin/Desktop/file2.txt<br></div><div>/Users/robin/Desktop/file3.txt<br></div></div><div><br></div><div><br></div><div>This is also what you can observe when running the manual testcase.</div><div><br></div><div>I have logged this in the bug database as issue JI-9041413.</div><div><br></div><div>Attached you find a webrev with the proposed patch.</div><div>The manual testcase succeeds with this patch, and fails without.</div><div>At the bottom of this email, you also find the output of hg diff.</div><div><br></div><div>The patch does not include a testcase, as there is already one available.</div><div><br></div><div>Regards,</div><div><br></div><div>Robin</div><div><br></div><div><br></div><div>hg diff output:</div><div><br></div><div><div>diff -r 0844fa517c35 src/macosx/classes/sun/lwawt/macosx/CDataTransferer.java</div><div>--- a/src/macosx/classes/sun/lwawt/macosx/CDataTransferer.java<span class="" style="white-space:pre"> </span>Tue Jul 05 11:03:13 2016 -0700</div><div>+++ b/src/macosx/classes/sun/lwawt/macosx/CDataTransferer.java<span class="" style="white-space:pre">       </span>Thu Jul 07 08:48:37 2016 +0200</div><div>@@ -148,6 +148,7 @@</div><div>             StringBuilder sb = new StringBuilder();</div><div>             if(strings.length > 0) {</div><div>                 sb.append(strings[0]);</div><div>+                sb.append(separator);</div><div>                 for(int i = 1; i < strings.length; i++) {</div><div>                     sb.append(strings[i]);</div><div>                     sb.append(separator);</div></div></div>