Previously, I duplicated the SignatureCacheHasher in the cuckoocache tests to avoid extra changes/review in the cuckoocache pr. This moves the SignatureCacheHasher to the sigcache header, out of the anonymous namespace, so that the tests can import it.
Not very critical, but it is good to make sure that these implementations don't ever diverge by only having this class defined once.
Everyone-loves-negative-net-diffs ACK 0b829c0
utACK 0b829c0
utACK
utACK but please don't mark this as trivial
utACK
utACK, checked moved/copied code is the same.
I reworded the commit because I had accidentally used trivial in it originally.
Code move not changed.
Needs rebase.
rebased
Please add an extra line between the commit title and the commit message, for some reason git likes to think that its all intended as a multi-line-super-long title if you dont.
This moves the SignatureCacheHasher to the sigcache header, out of the anonymous
namespace, so that the tests can import it.
@TheBlueMatt I think that's a local issue in your git client settings, I can't reproduce it. Nonetheless, I inserted an additional newline.
after-merge utACK f9c88079df8b78b8fab75e0903ca395654f2d360
