search

TeXitoi / structopt

Parse command line arguments by defining a struct.
Other
2.7k stars 149 forks source link

Update keyvalue.rs #502

Closed Eolien55 closed 2 years ago

Eolien55 commented 2 years ago

I updated keyvalue.rs to make it coherent with the actual behavior, since number_of_values is useless in that case.

TeXitoi commented 2 years ago

I don't get it, it's clearly explained that, if you don't number_of_values, it will break positional argument after.

Eolien55 commented 2 years ago

Yes, it is clearly explained, but after a bit of testing, it seems that it isn't true anymore. Maybe am I wrong, but positional arguments work just fine with almost the same code on my machine.

Eolien55 commented 2 years ago

Oh sorry, I didn't understand what you mean't by positional argument ; I get it now. Sorry again.

TeXitoi commented 2 years ago

No problem, if you have idea to clarify the text, don't hesitate to propose an improvement.