Renamed much for clarity

Does this make the code easier to understand?

What else appears to have a dubious name? I've been in and around this code for a few days now so it's harder for me to objectively judge this.
This function restores a single snapshot, so I think `allNodes` should change actually.
How about: `restoreSnapshot(signalGraphNodes, snapshot)`?