&CBlockIndex can be shared between threads safely, so it is sound to make CBlockIndex implement Sync.

Currently async code using CBlockIndex can't hold a & reference between await points, but actually this is fine, so we allow it.

There's no change in behavior.

cargo check