[8u-dev] Request for Approval: 8160941: [macosx]"text/uri-list" dataflavor concats the first two strings

[Claes Redestad]

Hi,

this seems wrong as it will add a comma at the end if there's only one item.

Shouldn't this rather be something like:

sb.append(strings[0]);
for(int i = 1; i < strings.length; i++) {
+                    sb.append(separator);
                      sb.append(strings[i]);
-                    sb.append(separator);
                  }

Thanks!

/Claes

On 2016-07-15 14:12, Mikhail Cherkasov wrote:
> Hi all,
>
> Could you please approve integration of the following fix:
>
> webrev: http://cr.openjdk.java.net/~rstevens/8160941/webrev.01/
> jbs: https://bugs.openjdk.java.net/browse/JDK-8160941
> review: 
> http://mail.openjdk.java.net/pipermail/awt-dev/2016-July/011642.html
>
> Thanks,
> Mikhail.