HI Robert,

Am Dienstag, den 21.02.2006, 01:36 +0100 schrieb Robert Schuster:
> Hi,
> I am surprised myself but I really got this working. :)
> 
> What I fixed here are mainly drawing problems. The underlying representation 
> of
> the highlighted area was mostly correct. There are some small glitches in the
> behavior which I am going to fix next.
> 
> Please comment the patch.

Very good, please commit.

/Roman

> 
> The ChangeLog:
> 
> 2006-02-21  Robert Schuster  <[EMAIL PROTECTED]>
> 
>         * javax/swing/plaf/basic/BasicTextUI.java:
>         (paint): Remove unneccessary part of the if-expression.
>         (damageRange): Added case where the range spans multiple lines.
>         * javax/swing/text/DefaultCaret.java:
>         (clearHighlight): New method.
>         (handleHighlight): Removed unneccessary part of the if-expression.
>         (setDot): Use clearHighlight method.
>         * javax/swing/text/DefaultHighlighter.java: Use ArrayList instead
>         of Vector.
>         (paint): Prevented calling size() on every loop iteration, fixed
>         calculation of allocation area bounds.
>         (getHighlights): Implemented.
>         (removeHighlight): Mark damaged area in textcomponent.
>         (addHighlight): Mark damaged area in textcomponent.
>         (changeHighlight): Mark damaged area in textcomponent.
>         (DefaultHighlighter.HighlightEntry): Made it a real
>         Highlighter.Highlight implementation.
>         (DefaultHighlighter.DefaultHighlightPainter.paint): Fixed
>         calculations.
> 
> cya
> Robert
> 

Attachment: signature.asc
Description: Dies ist ein digital signierter Nachrichtenteil

Reply via email to