Re: [cp-patches] FYI: javax/swing/JScrollPane.java

[Roman Kennke]

> OK. Found it. It was like Lillian already said, the viewport was not
> actually added in the constructor. The following patch makes sure it is
> and that the scrollbars actually work. It also avoids any state change
> events being fired in the constructor.
>
> 2005-06-22  Mark Wielaard  <address@hidden>
>
>       Fixes bug #13439
>         * javax/swing/JScrollPane.java (JScrollPane(Component,int,int)):
>         Set viewport, view and scrollLostener manually.
>
> With this applied ODB looks like Timo's screenshot:
> http://protu.it.helsinki.fi/~lindi/odb-classpath-2005-06-14T21:00:00+0000.png
>
> Does this patch look sane?

Looks sane and seems to work.

/Roman