Restructure IniDocument's constructors.

The one that takes a TextReader no longer closes it.
The one that opens a file from a path properly disposes its own StreamReader.
Some changes recommended by ReSharper.