<AWT Dev> [8] Review request JDK-8160941: [macosx]"text/uri-list" dataflavor concats the first two strings
Alexandr Scherbatiy
alexandr.scherbatiy at oracle.com
Wed Jul 13 08:29:36 UTC 2016
The fix looks good to me.
Thanks,
Alexandr.
On 7/12/2016 3:12 PM, Robin Stevens wrote:
> Hello Mikhail,
>
> thanks for the review.
> Would you mind performing the actual commit once it is approved (I
> assume I need a review from a second person) ?
> I have no commit rights, so I cannot do it myself.
>
> Thanks
>
> Robin
>
> On Tue, Jul 12, 2016 at 2:04 PM, Mikhail Cherkasov
> <mikhail.cherkasov at oracle.com <mailto:mikhail.cherkasov at oracle.com>>
> wrote:
>
> 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/20160713/e63c2306/attachment.html>
More information about the awt-dev
mailing list