docopt / docopt.cpp

C++11 port of docopt
Boost Software License 1.0
1.04k stars 146 forks source link

Should docopt::Options be an std::unordered_map? #154

Open mansoor2016 opened 2 years ago

mansoor2016 commented 2 years ago

Is the ordered nature of the options used?

mibli commented 2 years ago

Definietevely in run_testcase.cpp, but it could be changed, since performance of tests doesn't matter.

jaredgrubb commented 2 years ago

In retrospect maybe it should have been unordered_map but map was just chosen as default. I don't really see much benefit or issue either way -- I don't expect that most uses are going to have more than 10s of options, certainly not 10000s!