> IMO, this usage should just go, in preference to > > type_set_length (type, DW_UNSND (attr)); I tend to agree, but maybe for a different reason. I think we should eventually try to make struct type and main_type opaque, which will force to get rid of the macros and provide accessors instead. -- Joel