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 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.

-- 
Daniel Jacobowitz
CodeSourcery


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