Finalizer fix

[stesee]

• [/] You've read the Contributor Guide and Code of Conduct.
• [/] You've included unit or integration tests for your change, where applicable.
• [/] You've included inline docs for your change, where applicable.
• [/] There's an open issue for the PR that you are making. If you'd like to propose a change, please open an issue to discuss the change or find an existing issue.

Summary of the changes (Less than 80 chars)

Fixes #950

Description

Dropped the finalizer in IndexReader

[Shazwazza]

IMO formatting should be done in a separate PR. But that said, we should have correct indentation settings for more file types in .editorconfig and default settings for unspecified files. 2 spaces for XML/JSON type files seems to be the standard.

[NightOwl888]

Thanks for the PR and your willingness to help make our project better.

There are a few changes we would like you to make before we can accept this PR. Please see the comments inline.