Skip to content

[V132-specific; second of two] Current network manager tx.Hash_deprecated() to tx.Hash()

[V132-specific]

Intended to close #1938 (closed)
'common.Tx Hash() ignores Coins'.

Following !3499 (merged)'s merging, changing current-version Hash_deprecated() to Hash() can now (I believe) be done safely,
in turn allowing removal of the Hash_deprecated() function upon hard fork.

According to !3499 (diffs) the only current-version call of Tx Hash() is in the network manager;
if there were other calls, the pipeline would have been expected to fail from the renaming of their function
(which was why this was done in two parts, to ensure no new Hash() call prior to merging that could cause sync failure).

Edited by Multipartite

Merge request reports