[cp-patches] RFC: javax.swing.text.DefaultCaret

Carsten Neumann cn-develop at gmx.net
Wed Jun 21 17:58:17 UTC 2006


	Hi Mark,

Mark Wielaard wrote:
> Hi Carsten,
> 
> On Sat, 2006-03-25 at 16:48 +0100, Carsten Neumann wrote:
>> Mark Wielaard wrote:
>>> On Fri, 2006-03-24 at 23:52 +0100, Carsten Neumann wrote:
>>>
>>>> I've implemented the method: public boolean isActive().
>>> Nice. Useful method also. Last time I worked with the DefaultCaret I was
>>> confused there wasn't such a method. I see this was added in 1.5. Could
>>> you add a @since 1.5.
>> see attached new version of patch.
> 
> I see this was never applied, sorry about that.
> Could you take a look at DefaultCaret and see if it is still the correct
> way to implement this? Since you submitted the patch Robert added some
> new logic and an active field to that class.

now it's my turn to apologize for the long delay in answering your
request :-/
In answer to your question, I do believe the patch implements what the
API doc of the RI specifies, but I'm not sure if that's also what the RI
does ;)
Anyway, I updated the patch, so it is now against current cvs. I need to
finish my diploma thesis (dunno if that's the right translation) at the
moment, so I don't have the time to check the RI behaviour, sorry.

	Cheers,
		Carsten
-------------- next part --------------
A non-text attachment was scrubbed...
Name: javax.swing.text.DefaultCaret-2006-06-21.diff
Type: text/x-patch
Size: 1087 bytes
Desc: not available
Url : http://developer.classpath.org/pipermail/classpath-patches/attachments/20060621/04fd8826/javax.swing.text.DefaultCaret-2006-06-21.bin


More information about the Classpath-patches mailing list