Issue 499

[d-kad]

Solves #499 by fixing the right order of source and destination nodes in copy_tree and move_tree methods.
Adds a test comparing subtrees after successful copy_tree.

[tturocy]

I'm not following the intent of DeleteOutcome. The source node shouldn't be changed.. But further, DeleteOutcome does not just un-set the outcome at the node, it completely removes the outcome from the game - which we definitely don't want!

For this because it's a bug that's existed since 16.2, we want to base this on and merge this into maint16_2 - from there it will get merged forward into newer versions.

Finally, don't forget to add a ChangeLog entry (as this is a user-visible change) and mentioning the issue number.

[tturocy]

Merged at:

• dab6a39 in maint16_2
• 0bb6ddf in maint16_3
• e487cb4 in master