`Indexer::index_tx` can have a default impl in the trait.

Open evanlinjin opened this issue 2 years ago • 2 comments

          I think this is a good idea. Also `index_tx` can have a default impl in the trait.

Originally posted by @LLFourn in https://github.com/bitcoindevkit/bdk/pull/1084#discussion_r1305441253

Aug 26 '23 12:08 evanlinjin

@evanlinjin as discussed, I'm taking this up.

Oct 12 '23 15:10 vladimirfomene

See https://github.com/bitcoindevkit/bdk/pull/1176#issuecomment-1928183948 why I no longer think this is a good idea.

Feb 05 '24 22:02 LLFourn