Hi,

I see that the change introduced the base abstraction class, X11InputMethodBase. Back in the days I worked on it, X11InputMethod is the base abstract class, and platform impls. have names representing them, i.e, MInputMethod for Motif, and XInputMethod for XAWT. Now X11InputMethod is a platform dependent implementation. It'd be a bit clearer to use the different impl class names.

Naoto

On 5/30/18 1:27 PM, Langer, Christoph wrote:
Hi,

I was just asked to include i18-n dev in the review of the change for this Item.
Bug: https://bugs.openjdk.java.net/browse/JDK-8201429

Last webrev: http://cr.openjdk.java.net/~clanger/webrevs/8201429.3/

Unfortunately this request came a bit too late and the change has already been 
pushed: http://hg.openjdk.java.net/jdk/client/rev/668463f93ec0

So this is now merely a notification. However, maybe you can still have an eye 
on this or even do some specific testing. In case you see some issue with it, 
we shall open a follow-up item.

Thanks
Christoph

Reply via email to