Patch: add ActionEvent and TextEvent handling to TextComponent peers

Thomas Fitzsimmons fitzsim@redhat.com
Mon Jul 7 19:57:00 GMT 2003


On Sat, 2003-07-05 at 01:20, Tom Tromey wrote:
> 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).
> 

Patch checked in with requested fixes.

Tom

-- 
Thomas Fitzsimmons <fitzsim@redhat.com>



More information about the Java-patches mailing list