The change here is intended to give "Reindexing aborted" in the log (nothing more in the end, as I found it weird to read ``Reindexing finished`` after a crash).

I understand your wish to link setting fReindex with writing the reindex-state to block index. But if fRequestShutdown is getting true we also don't write it?