accumulator/utils: remove unused function `subTreePositions`

mit-dci / utreexo

accumulator for bitcoin utxo set

MIT License

323 stars 60 forks source link

accumulator/utils: remove unused function `subTreePositions` #354

Closed theStack closed 2 years ago

theStack commented 2 years ago

This function hasn't been used since commit e83f377a2a6f9c3287ef64d01eab1e02b3c27606. It seems that it was only useful for removev2 and earlier, i.e. it makes sense to get rid of it.

kcalvinalvin commented 2 years ago

ACK 1a5f7d17d41a096fd9ac1e290e545b4c86303d43

We're moving towards a non-swapping accumulator algorithm too it makes even more sense to get rid of this.

adiabat commented 2 years ago

Looks good. Yeah in the swapless branch lots of these types of functions are getting removed.