Hi Simon,

> > +/* Given TYPE, which is a fixed point type, return its scaling factor.  */
> > +extern const gdb_mpq &fixed_point_scaling_factor (struct type *type);
> 
> I think this would belong in fixed_point_type_info, so you'd access it
> using:
> 
>   type->fixed_point_info ().scaling_factor ()

In pratice, this wouldn't work. The main reason-d'etre for this function
is that fixed point types can be both a TYPE_CODE_FIXED_POINT as well
as a TYPE_CODE_RANGE of a TYPE_CODE_FIXED_POINT. What the function
above does is first call fixed_point_type_base_type in order to get
to the type which actually has the fixed_point_info, and then from
there accesses the scaling factor.

What we can do is make that a method of struct type instead...
This is what the attached patch does.

-- 
Joel