akamsteeg/AtleX.CommandLineArguments

PrefixedKeyParser is abstract but shouldn't be

Closed this issue · 0 comments

Currently, PrefixedKeyParser is an abstract class:

However, I don't think that it should be. It's a complete implementation of ICommandLineArgumentsParser that can work on its own. Now we're forcing people to extend it, just to call the constructor of the abstract base class PrefixedKeyParser . It's clean, because it forces strong-typed parsers, but it also makes introducing a new one for a 3rd-party thing much more involved.

Suggestion: Remove the abstract modified from PrefixedKeyParser .

This is related to #45