Hi, Petr.
A few comments:
1) if (fDropTarget != null)
1065 System.err.println("CComponent.addDropTarget(): current
drop target is non-null.");
Should we throw an exception instead or dispose the previous drop target?
2) We can create/dispose DT on any thread?
3) {@inheritDoc} is obvious if no additional information is added.
On 4/30/14 5:33 PM, Petr Pchelko wrote:
Hello, AWT Team.
Please review a simple cleanup fix for the issue:
https://bugs.openjdk.java.net/browse/JDK-8042219
The fix is available here:
http://cr.openjdk.java.net/~pchelko/9/8042219/webrev/
We should not reference macosx classes in lwawt package. I've also cleaned up
CDropTarget class a bit.
Thank you.
With best regards. Petr.
--
Best regards, Sergey.
