<AWT Dev> [8] Review request JDK-8160941: [macosx]"text/uri-list" dataflavor concats the first two strings

Mikhail Cherkasov mikhail.cherkasov at oracle.com
Tue Jul 12 12:04:37 UTC 2016


Looks good.

On 12.07.2016 0:15, Robin Stevens wrote:
> Hello Mikhail,
>
> good catch. I adjusted the code.
>
> Webrev: http://cr.openjdk.java.net/~rstevens/8160941/webrev.01 
> <http://cr.openjdk.java.net/%7Erstevens/8160941/webrev.01>
>
>
> Robin
>
> On Mon, Jul 11, 2016 at 6:08 PM, Mikhail Cherkasov 
> <mikhail.cherkasov at oracle.com <mailto:mikhail.cherkasov at oracle.com>> 
> wrote:
>
>     On 11.07.2016 18:55, Robin Stevens wrote:
>
>         However, that would cause an inconsistency with other
>         operating systems.
>
>     ok, but then let's simplify it:
>
>     remove this line:
>          sb.append(strings[0]);
>
>     and start cycle from 0:
>     for(int i = 0; i < strings.length; i++) {
>        sb.append(strings[i]);
>        sb.append(separator);
>     }
>
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.openjdk.java.net/pipermail/awt-dev/attachments/20160712/0145ad0b/attachment.html>


More information about the awt-dev mailing list