[sothatsit]

I think you also need to handle TypeInsnNodes, maybe even LdcInsnNodes as well for(Types).
I'd recommend you look at the code I posted to see the sort of things you should refactor and what you shouldn't (which insns, which methods, hierarchy methods and classes etc)

Sorry for the late reply, been really busy. I have been looking through AbstractInsnNode's sub classes and it seems these are the ones which refer to types which could be refactored:
TypeInsnNode
MethodInsnNode
MultiANewArrayInsnNode
FieldInsnNode

Those are the ones which refer to names which could be refactored.

On another note, i have started to switch from visitors to using ClassNodes. There is a performance difference but its not much of an issue for me.

Thanks for all your help with everything!