Hello, Anthony. Thank you for the review.
> And I assume it also reverses JDK-6699328 ? In a certain way. I’ve deleted all the code where that fix was made. With best regards. Petr. > On Jun 23, 2014, at 9:59 PM, Anthony Petrov <anthony.pet...@oracle.com> wrote: > > The fix looks fine to me. > >> this fix is a reverse-changeset for JDK-6638195 > > And I assume it also reverses JDK-6699328 ? > > -- > best regards, > Anthony > > On 6/23/2014 6:40 PM, Petr Pchelko wrote: >> Hello, Anthony. >> >> Thank you for the review. >> I've checked that this fix is a reverse-changeset for JDK-6638195 where the >> class was introduced. >> >> With best regards. Petr. >> >> On 23 июня 2014 г., at 18:32, Artem Ananiev <artem.anan...@oracle.com> wrote: >> >>> Hi, Petr, >>> >>> the fix looks fine to me. >>> >>> I would also suggest you to find the initial fix, when EventQueueDelegate >>> was introduced (in 6u10?), to check if anything else can be wiped out. >>> >>> Thanks, >>> >>> Artem >>> >>> On 6/23/2014 6:13 PM, Petr Pchelko wrote: >>>> Hello, AWT Team. >>>> >>>> Please review the fix for the issue: >>>> https://bugs.openjdk.java.net/browse/JDK-8047798 >>>> The fix is available at: >>>> http://cr.openjdk.java.net/~pchelko/9/8047798/webrev/ >>>> >>>> One more weekly clenup. The EventQueueDelegate was added for JavaFXScript >>>> to manipulate EDT in unconventional ways. >>>> Now it's not used and can be removed. Grepped the sources for the >>>> raminings and built a JPRT. >>>> >>>> With best regards. Petr. >>>> >>
