Patch: add ActionEvent and TextEvent handling to TextComponent peers

Tom Tromey tromey@redhat.com
Sat Jul 5 05:29:00 GMT 2003


Tom> I think it is best to use the more general approach, where possible,
Tom> because it can potentially reduce the amount of native code, and make
Tom> the native code that remains more "object-oriented."

Thanks for looking into this.  Your analysis is great.

Tom> Yes, I should have mentioned this; the setEditable and setCaretPosition
Tom> already have working native implementations.  It's just that they were
Tom> being overridden by non-working Java implementations, for whatever
Tom> reason.

It may be my fault.  I'm guessing I made this change and then forgot
to remove the native methods.  Not sure though.

With the indentation fixlets the patch is fine to check in.  I can put
it in Classpath if you want; just send me the updated patch (privately
is fine).

Tom



More information about the Java-patches mailing list