docopt/docopt.cpp

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

mansoor2016 opened this issue · 2 comments

mansoor2016 commented

Is the ordered nature of the options used?

mibli commented

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

jaredgrubb commented

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!

  • 👍1