Re: [9] Review Request: 8047336 Read flavormap.properties as resource

2014-07-09 Thread Petr Pchelko
up the HTML part of the email (see > below a text-only quote), so here's the correct link to the webrev: > > http://cr.openjdk.java.net/~pchelko/9/8047336/webrev.01/ > > The fix looks fine to me. > > -- > best regards, > Anthony > > On 7/2/2014 3:10 PM, Pe

Re: [9] Review Request: 8047336 Read flavormap.properties as resource

2014-07-02 Thread Petr Pchelko
st regards. Petr. On 02 июля 2014 г., at 13:52, Alan Bateman wrote: > On 01/07/2014 09:35, Petr Pchelko wrote: >> Hello, >> >> The changes in the public API have been approved, so let me continue the >> review process. >> >> For your convenience: >

Re: [9] Review Request: 8047336 Read flavormap.properties as resource

2014-07-01 Thread Petr Pchelko
., at 15:50, Alan Bateman wrote: > On 20/06/2014 12:41, Petr Pchelko wrote: >> Hello, Anthony, Artem. >> >>>> Do we officially declare that we drop support for this possibility? >>> This possibility will be dropped regardless of the current Petr's fix, >&g

Re: [9] Review Request: 8047336 Read flavormap.properties as resource

2014-06-20 Thread Petr Pchelko
, the current fix is not about flavormap.properties on its own, but about > removing AWT.DnD.flavorMapFileURL toolkit property. I would suggest to push > this change as a separate bug fix, not as a part of 8047336. > > Thanks, > > Artem > >> -- >> best regards, >

Re: [9] Review Request: 8047336 Read flavormap.properties as resource

2014-06-19 Thread Petr Pchelko
Petr. On 19 июня 2014 г., at 16:13, Alan Bateman wrote: > On 19/06/2014 12:17, Petr Pchelko wrote: >> Hello, >> >> Please review the fix for the issue: >> https://bugs.openjdk.java.net/browse/JDK-8047336 >> The fix is available at: >> http://cr.openjdk.j

[9] Review Request: 8047336 Read flavormap.properties as resource

2014-06-19 Thread Petr Pchelko
Hello, Please review the fix for the issue: https://bugs.openjdk.java.net/browse/JDK-8047336 The fix is available at: http://cr.openjdk.java.net/~pchelko/9/8047336/webrev.00/ This is another step in datatransfer modularization work. This part of the work needs a CCC, so I've moved it out to a s