[
Date Prev][
Date Next][
Thread Prev][
Thread Next][
Date Index][
Thread Index]
[
List Home]
|
Re: [cdt-dev] Re: [cdt-patch] Register View Persistence
|
----- Original Message -----
From: "Chris Songer" <songer@xxxxxxxxxxxxx>
To: <cdt-dev@xxxxxxxxxxx>
Cc: <cdt-patch@xxxxxxxxxxx>
Sent: Friday, April 04, 2003 5:34 PM
Subject: [cdt-dev] Re: [cdt-patch] Register View Persistence
> At 04:40 PM 4/4/2003 -0500, Mikhail Khodjaiants wrote:
> > And
> >this preserves only the first level structure of the register tree.
>
> Moving to dev.
>
> Ah, I see the extra level now.
>
> In practice I've certainly never seen this created and I don't have a
> testcase for it. I don't have any faith what-so-ever that I'd get it right
> without a way to test it. If you have any suggestions for how to set
> switches so that CRegister returns true to getValue().hasVariables(), I'd
> be happy to add additional level support if only because once our testing
> folks find those switches they will be asking for it anyway.
The possible implementation is to save the state per target. The only
problem is performance but we can make it optional. I'll discuss this with
Alain and Dave on Monday.
>
> Thanks!
> -Chris
>
>
>
> >Mikhail
> >----- Original Message -----
> >From: "Chris Songer" <songer@xxxxxxxxxxxxx>
> >To: <cdt-patch@xxxxxxxxxxx>
> >Sent: Friday, April 04, 2003 1:20 PM
> >Subject: [cdt-patch] Register View Persistence
> >
> >
> > >
> > > Hi,
> > >
> > > Step through a function return and see the open groups in your
register
> > > view close. That seems to be because the IStackFrame is different,
even
> > > though the underlying register model is the same. (Still, the register
> > > model could change I suppose.) Our usability testers hated it.
> > >
> > > So, please find attached a patch that causes the register view to
remember
> > > what register groups are expanded and restore them to their expanded
state
> > > across stackframe changes. The patch was based on 1.0.1. It looks to
me
> > > like it will cleanly apply to the head, but your mileage may vary.
> > >
> > > Thanks!
> > > -Chris
> >
> >_______________________________________________
> >cdt-patch mailing list
> >cdt-patch@xxxxxxxxxxx
> >http://dev.eclipse.org/mailman/listinfo/cdt-patch
>
> _______________________________________________
> cdt-dev mailing list
> cdt-dev@xxxxxxxxxxx
> http://dev.eclipse.org/mailman/listinfo/cdt-dev
>
