Configure INI parser to support bethesda ini files

[Al12rs]

fixes #448

Configuration taken from Mutagen:
https://github.com/Noggog/Mutagen/blob/369a0549fab90593cb130b3889f988228f8ecf98/Mutagen.Bethesda.Core/Archives/DI/GetArchiveIniListings.cs#L38-L46

Tested with ENB for Legendary edition.

[erri120]

Should probably add some tests for this.

[Al12rs]

Should probably add some tests for this.

Something like installing a mod with a "bad" ini file?

[erri120]

Should have some tests for the config.

[erri120]

Should probably add some tests for this.

Something like installing a mod with a "bad" ini file?

You can make the settings object internal and create a test that has various ini files as input and just test if that works:

void Test_IniParser()
{
  const string ini = @"
# this is a comment
foo = bar
";
  //...
}

[Al12rs]

That assumes I know all the "interesting" cases for inis to fail. What are we checking for exactly?
Is this for the goal of regression testing or more to see if we are covering all the cases we want?

[erri120]

That assumes I know all the "interesting" cases for inis to fail. What are we checking for exactly? Is this for the goal of regression testing or more to see if we are covering all the cases we want?

On main, the INI analyzer fails because it can't deal with a comment (#448). You can create a test with an INI file that contains a comment, which should fail on main and succeed on your branch.

[Al12rs]

Added a test for this