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

[Lillian Angel]

On Wed, 2005-06-22 at 16:24 +0200, Mark Wielaard wrote:
> Hi,
> 
> On Wed, 2005-06-22 at 15:36 +0200, Mark Wielaard wrote:
> > With my patch reapplied that issue goes away. And IDB starts up again.
> > But according to bug #13439 "swing: regression with omniscient debugger"
> > https://savannah.gnu.org/bugs/?func=detailitem&item_id=13439
> > this isn't a full solution since IF YOU PATCH THE APPLICATION you can
> > get it to show more stuff without my patch applied. I am not clear on
> > why that is though. And I think we should not need application patching.
> 
> 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?

This patch works great for me. 

> Cheers,
> 
> Mark
> _______________________________________________
> Classpath-patches mailing list
> address@hidden
> http://lists.gnu.org/mailman/listinfo/classpath-patches