On Tue, 2008-01-08 at 13:19 -0200, Thiago Jung Bauermann wrote:
> param_ptr should be a const gdb_byte *, and the cast here should be
> removed. In addition, I believe the use of value_offset here is wrong.
> Other uses of value_offset are to obtain a memory address in the
> inferior address space, not in GDB's. Also, I think that value_contents
> should be used intead of value_contents_all so that the value's
> embedded_offset can be taken into account.
> 
> > +	      dfp_ptr = (gdb_byte *) value_contents_all (dfp_value)
> > +				      + value_offset (dfp_value);
> 
> Same here.

Fixed.

-- 
Luis Machado
Software Engineer 
IBM Linux Technology Center