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.
