commandline icon indicating copy to clipboard operation
commandline copied to clipboard

Removed where T : new()

Open CreepyGnome opened this issue 8 years ago • 3 comments

Removed where T : new() from the ParseArguments<T> factory method as it shouldn't be required if convention is followed of only mutable classes required a default constructor, and immutable classes are supported and do not have this requirement, so the new() restriction is blocking immutable classes which should typically not have a default constructor.

This resolves #310

CreepyGnome avatar Jun 15 '16 17:06 CreepyGnome

can we merge this please been over a year waiting on this fix

clmcgrath avatar Jul 10 '17 19:07 clmcgrath

Note: We have moved the project to a more team managed repository. Please resubmit any pull requests to https://github.com/commandlineparser/commandline Sorry for inconveniences.

ericnewton76 avatar Nov 04 '17 14:11 ericnewton76

You had over a year to deal with this PR, long before you created a new repo. If you still have this issue and still would like to address it feel free to just copy and paste the changes where needed or reimplement your own solution.

CreepyGnome avatar Nov 05 '17 00:11 CreepyGnome