Hello, I've a patch (well a sequence of patches) that replace ``value_ptr'' with ``struct value *''. Can anyone think of a reason to not treat this as an obvious (if slightly anal) cleanup? Andrew