added workaround for if a StandardAttributes is saved with an incoming connection to attributes->visibility->name
Optimised ComputeNode::hash().

The call to relativeName() was expensive because it was allocating memory. It's quicker to just iterate up the hierarchy ourselves and directly append each name we find along the way. I had tried optimising this in the past, but never got any measurements that confirmed a speedup. Now we've got much superior performance from temporary Contexts, the relativeName() cost comes through more clearly and there are obvious measurable benefits. A simple Constant->Reformat benchmark shows ~14% reduction in runtime, and a SceneReader benchmark shows a small but measurable improvement.

Also added some documentation for why the hash is computed the way it is.
Merge pull request #858 from davidsminor/standardAttributesIncomingConnection

StandardAttributes incoming connection workaround
Merge pull request #859 from johnhaddon/hashOptimisation

Optimised ComputeNode::hash().