This is the mail archive of the gdb-patches@sourceware.org mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: Remove deprecated_set_value_type


On 13/11/2007, Daniel Jacobowitz <drow@false.org> wrote:
> On Tue, Nov 13, 2007 at 10:01:11AM +0000, Rob Quill wrote:
> > Hi,
> >
> > This patch removes the deprecated_set_value_type() function from
> > value.c, value.h and all other places which use it.
> >
> > I also plan to try and remove the other deprecated functions from value.c
>
> Sorry, but you've missed the point.  It was deprecated because you're
> not supposed to bang on a value's type after it is created (or at
> least, that's the hope).  And struct value was moved from the header
> to the C file in order to stop other places from poking at it.
>
> So this is going the wrong direction :-(
>
> > -/* This is being used to change the type of an existing value, that
> > -   code should instead be creating a new value with the changed type
> > -   (but possibly shared content).  */
>
> As that says.

Ah, ok, I thought it seemed a bit simple. I'll try and fix it properly now :)

Rob


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]