This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
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