It is in a platform dependent directory .. and I gather this allows the build to pick up the right one
without any work here.

-phil

On 5/30/2018 2:08 PM, Naoto Sato wrote:
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