I would rather see char* casting done only at the external API layer -- APIs we have control over should be constified when possible, IMO.