alliedmodders / amtl

AlliedModders C++ Template Library
29 stars 19 forks source link

Remove BoolOption from argparser? #89

Closed Fyren closed 4 years ago

Fyren commented 4 years ago

BoolOption is broken because since it can take true/false, so it tries to eat the next thing on the command line. If it fails, because the next thing is just another parameter or whatever, it just leaves the option's value in the default state. So ./cmd --bool-option positional with a default of false means that option stays false, the same as if --bool-option wasn't given.

Can probably just kill it off in favor of ToggleOption/EnableOption.

dvander commented 4 years ago

Removed.